From 004f81f8d1ec38047dba4d198311d31ccc8dd94d Mon Sep 17 00:00:00 2001 From: David Chisnall Date: Mon, 11 Jan 2010 22:33:19 +0000 Subject: [PATCH] Allow VLAs in C++ if in GNU mode (GNU C++ permits them). Clang can now compile LanguageKit, although the resulting code crashes (although not in any of the functions that use VLAs). git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@93198 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Sema/SemaType.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/Sema/SemaType.cpp b/lib/Sema/SemaType.cpp index 2bddf9ecd6..0f9bd516b4 100644 --- a/lib/Sema/SemaType.cpp +++ b/lib/Sema/SemaType.cpp @@ -651,8 +651,10 @@ QualType Sema::BuildArrayType(QualType T, ArrayType::ArraySizeModifier ASM, } T = Context.getConstantArrayType(T, ConstVal, ASM, Quals); } - // If this is not C99, extwarn about VLA's and C99 array size modifiers. - if (!getLangOptions().C99) { + // If this is not C99 or C++ with GNU extenisons, extwarn about VLA's and C99 + // array size modifiers. + if (!getLangOptions().C99 && + !(getLangOptions().CPlusPlus && getLangOptions().GNUMode)) { if (ArraySize && !ArraySize->isTypeDependent() && !ArraySize->isValueDependent() && !ArraySize->isIntegerConstantExpr(Context)) -- 2.50.1