[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-09-08 Thread bergner at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

--- Comment #10 from Peter Bergner  2011-09-08 
18:09:02 UTC ---
Author: bergner
Date: Thu Sep  8 18:08:53 2011
New Revision: 178703

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=178703
Log:
gcc/
Backport from mainline

2011-08-23  Jakub Jelinek  

PR c++/46862
* class.c (finish_struct_1): If TYPE_TRANSPARENT_AGGR is set on a type
which doesn't have any fields, clear it and diagnose.

2011-08-29  Jakub Jelinek  
Jason Merrill  

PR c++/50207
* class.c (finish_struct_1): Complain if the first field is
artificial.

gcc/testsuite/
Backport from mainline
2011-08-23  Jakub Jelinek  

PR c++/46862
* g++.dg/dfp/nofields.C: New test.

2011-08-29  Jakub Jelinek  
Jason Merrill  

PR c++/50207
* g++.dg/dfp/base.C: New test.

Added:
branches/ibm/gcc-4_5-branch/gcc/testsuite/g++.dg/dfp/base.C
branches/ibm/gcc-4_5-branch/gcc/testsuite/g++.dg/dfp/nofields.C
Modified:
branches/ibm/gcc-4_5-branch/gcc/ChangeLog.ibm
branches/ibm/gcc-4_5-branch/gcc/cp/class.c
branches/ibm/gcc-4_5-branch/gcc/testsuite/ChangeLog.ibm


[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-08-30 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

--- Comment #9 from Jason Merrill  2011-08-30 
21:48:38 UTC ---
Author: jason
Date: Tue Aug 30 21:48:34 2011
New Revision: 178343

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=178343
Log:
PR c++/50207
* class.c (finish_struct_1): Complain if the first field is
artificial.

Added:
branches/gcc-4_6-branch/gcc/testsuite/g++.dg/dfp/base.C
Modified:
branches/gcc-4_6-branch/gcc/cp/ChangeLog
branches/gcc-4_6-branch/gcc/cp/class.c
branches/gcc-4_6-branch/gcc/testsuite/ChangeLog


[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-08-29 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

Jason Merrill  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #8 from Jason Merrill  2011-08-30 
04:37:23 UTC ---
Fixed.


[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-08-29 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

--- Comment #7 from Jason Merrill  2011-08-30 
04:30:56 UTC ---
Author: jason
Date: Tue Aug 30 04:30:42 2011
New Revision: 178276

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=178276
Log:
PR c++/50207
* class.c (finish_struct_1): Complain if the first field is
artificial.

Added:
trunk/gcc/testsuite/g++.dg/dfp/base.C
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/class.c
trunk/gcc/testsuite/ChangeLog


[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-08-29 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

Jason Merrill  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|unassigned at gcc dot   |jason at gcc dot gnu.org
   |gnu.org |

--- Comment #6 from Jason Merrill  2011-08-29 
14:59:43 UTC ---
The problem here is not that the first field is aggregate, but that the first
field is a fake field for the base class.  I'll adjust the patch accordingly.


[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-08-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

--- Comment #5 from Jakub Jelinek  2011-08-29 
12:48:10 UTC ---
Created attachment 25125
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25125
gcc47-pr50207.patch

Untested patch which adds that limitation.   still seems to
work.


[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-08-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jason at gcc dot gnu.org

--- Comment #4 from Jakub Jelinek  2011-08-29 
11:18:10 UTC ---
I'd say this is invalid, TYPE_TRANSPARENT_AGGR is an ugly hack and we should
perhaps limit it just to scalar fields.


[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-08-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

--- Comment #3 from Jakub Jelinek  2011-08-29 
10:05:34 UTC ---
Oops, sorry, that was for PR50208.


[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-08-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

Jakub Jelinek  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2011-08-29
 CC||jakub at gcc dot gnu.org
 Ever Confirmed|0   |1

--- Comment #2 from Jakub Jelinek  2011-08-29 
10:04:09 UTC ---
Caused by http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=177647


[Bug c++/50207] [4.7 Regression] G++ segv's on reduced test case

2011-08-27 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50207

Richard Guenther  changed:

   What|Removed |Added

   Target Milestone|--- |4.7.0
Summary|G++ segv's on reduced test  |[4.7 Regression] G++ segv's
   |case|on reduced test case