Re: C++ PATCH for c++/68703 (dependent vector length)

2016-09-29 Thread Jason Merrill
On Wed, Sep 28, 2016 at 2:43 PM, David Edelsohn wrote: > Hi, Jason > > This patch added testcases g++.dg/ext/vector32.C and vector32a.C, but > there is no gcc/testsuite/ChangeLog entry. > > The testcases are failing on AIX with a strange ICE: > > src/gcc/testsuite/g++.dg/ext/vector32.C: In functio

Re: C++ PATCH for c++/68703 (dependent vector length)

2016-09-28 Thread David Edelsohn
Hi, Jason This patch added testcases g++.dg/ext/vector32.C and vector32a.C, but there is no gcc/testsuite/ChangeLog entry. The testcases are failing on AIX with a strange ICE: src/gcc/testsuite/g++.dg/ext/vector32.C: In function 'int main()': src/gcc/testsuite/g++.dg/ext/vector32.C:18:1: interna

Re: C++ PATCH for c++/68703 (dependent vector length)

2016-08-09 Thread Jason Merrill
On Tue, Aug 9, 2016 at 6:11 PM, Jakub Jelinek wrote: > This test is the same as the other one. Did you mean to use > typedef int V __attribute__((vector_size (N * sizeof (int; > V v; > instead or something similar (where the attribute would live on > the type rather than on the FIELD_DECL

Re: C++ PATCH for c++/68703 (dependent vector length)

2016-08-09 Thread Jakub Jelinek
On Tue, Aug 09, 2016 at 06:02:45PM -0400, Jason Merrill wrote: > diff --git a/gcc/testsuite/g++.dg/ext/vector32.C > b/gcc/testsuite/g++.dg/ext/vector32.C > new file mode 100644 > index 000..8901d0b > --- /dev/null > +++ b/gcc/testsuite/g++.dg/ext/vector32.C > @@ -0,0 +1,18 @@ > +// PR c++/6870

C++ PATCH for c++/68703 (dependent vector length)

2016-08-09 Thread Jason Merrill
When we defer dependent attributes, we need to take that into account for determining whether something is dependent. Furthermore, such an attribute means that the TREE_TYPE is incomplete, so we shouldn't use it at all. Tested x86_64-pc-linux-gnu, applying to trunk. commit 931555dc7ab081ea7af1e4b