On 14 April 2016 at 17:26, Martin Sebor <mse...@gmail.com> wrote: > On 04/14/2016 04:39 AM, Andreas Schwab wrote: >> >> Martin Sebor <mse...@gmail.com> writes: >> >>> diff --git a/gcc/testsuite/g++.dg/cpp1y/vla11.C >>> b/gcc/testsuite/g++.dg/cpp1y/vla11.C >>> new file mode 100644 >>> index 0000000..af9624a >>> --- /dev/null >>> +++ b/gcc/testsuite/g++.dg/cpp1y/vla11.C >>> @@ -0,0 +1,711 @@ >>> +// PR c++/69517 - [5/6 regression] SEGV on a VLA with excess initializer >>> +// elements >>> +// PR c++/70019 - VLA size overflow not detected >>> +// >>> +// Runtime test to verify that attempting to either construct a VLA with >>> +// erroneous bounds, or initialize one with an initializer-list that >>> +// contains more elements than the VLA's non-constant (runtime) bounds >>> +// causes an exception to be thrown. Test also verifies that valid >>> +// VLAs and their initializers don't cause such an exception. >>> + >>> +// { dg-do run { target c++11 } } >>> +// { dg-additional-options "-Wno-vla" } >> >> >> On m68k: >> >> /daten/aranym/gcc/gcc-20160414/gcc/testsuite/g++.dg/cpp1y/vla11.C: In >> instantiation of 'struct TestType<32u>': >> /daten/aranym/gcc/gcc-20160414/gcc/testsuite/g++.dg/cpp1y/vla11.C:201:1: >> required from here >> /daten/aranym/gcc/gcc-20160414/gcc/testsuite/g++.dg/cpp1y/vla11.C:89:27: >> error: requested alignment 32 is larger than 16 [-Wattributes] > > > Thank you for the heads up (and sorry about the breakage). I've > committed r234976 to fix that. > Hi,
In your follow-up commit r234981, your gcc/testsuite/ChangeLog entry says that you reverted vla11.C, but the commit does not actually modify this testcase. As a matter of fact, I see it failing on arm and aarch64. Did you forget to remove it, or did you expect r234976 to fix it? Christophe. > Martin