[PATCH] D18823: Implementation of VlA of GNU C++ extension

[Dmitry Polukhin via cfe-commits]

DmitryPolukhin added inline comments.

================
Comment at: llvm/tools/clang/test/SemaCXX/c99-variable-length-array.cpp:1
@@ -1,1 +1,2 @@
-// RUN: %clang_cc1 -fsyntax-only -verify -Wvla-extension %s
+// RUN: %clang_cc1 -fsyntax-only -std=c++98 -DCPP98 -verify -Wvla-extension %s
+// RUN: %clang_cc1 -fsyntax-only -std=gnu++98 -DGNU98 -verify -Wvla-extension %s
----------------
rengolin wrote:
> are we not supporting this in c99 anymore?
No, it is C++ test due to file extension .cpp. So C99 mode shouldn't be affected as far as I understand (i.e. the test was never run in C99). Mode by default is gnu++98 where this patch enable VLA so some warnings/errors from this test are gone (-Wvla-extension).


http://reviews.llvm.org/D18823