[PATCH] Sema: Treat zero sized arrays as flexible array members

Richard Smith richard at metafoo.co.uk
Mon Sep 29 12:21:58 PDT 2014


Generally, I think this is a good direction, but seems large enough to deserve a release notes change. Have you tested this against much existing code?

================
Comment at: test/SemaCXX/zero-length-arrays.cpp:13
@@ -13,5 +12,3 @@
   int foo_count;
-  Foo foos[0];
-  Foo foos2[0][2];
-  Foo foos3[2][0];
+  Foo foos[0]; // expected-error-re {{flexible array member {{.*}} with non-trivial destruction}}
 
----------------
Can we find out what rdar://problem/10228639 was?

http://reviews.llvm.org/D5478






More information about the cfe-commits mailing list