From 394bf0d0ba9f5b89ce5fdfa1f024e92a2deada22 Mon Sep 17 00:00:00 2001 From: David Chisnall Date: Mon, 11 Jan 2010 22:33:19 +0000 Subject: 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). llvm-svn: 93198 --- clang/lib/Sema/SemaType.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'clang/lib/Sema') diff --git a/clang/lib/Sema/SemaType.cpp b/clang/lib/Sema/SemaType.cpp index 2bddf9ecd60..0f9bd516b49 100644 --- a/clang/lib/Sema/SemaType.cpp +++ b/clang/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)) -- cgit v1.2.3