[Bug middle-end/42837] [4.5 Regression] FAIL: g++.dg/abi/packed1.C execution test

2010-02-18 Thread jason at gcc dot gnu dot org


--- Comment #7 from jason at gcc dot gnu dot org  2010-02-19 01:16 ---
Subject: Bug 42837

Author: jason
Date: Fri Feb 19 01:16:28 2010
New Revision: 156885

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156885
Log:
PR c++/42837
* class.c (create_vtable_ptr): Set DECL_PACKED if type is packed.

Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/class.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/abi/packed1.C


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42837



[Bug middle-end/42837] [4.5 Regression] FAIL: g++.dg/abi/packed1.C execution test

2010-02-18 Thread jason at gcc dot gnu dot org


-- 

jason at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |jason at gcc dot gnu dot org
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2010-02-16 18:48:29 |2010-02-18 20:46:39
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42837



[Bug middle-end/42837] [4.5 Regression] FAIL: g++.dg/abi/packed1.C execution test

2010-02-17 Thread sje at cup dot hp dot com


--- Comment #6 from sje at cup dot hp dot com  2010-02-17 18:03 ---
I tried the test with GCC 4.4.0 and 4.3.3 on IA64, in those cases the test
failed because we didn't actually pack anything and then the if test failed and
we called abort.

$ /opt/hp-gcc-4.3.3/bin/g++ -Wpacked packed1.C -o x;./x   
packed1.C:7: warning: packed attribute causes inefficient alignment for 'INNER'
packed1.C:13: warning: ignoring packed attribute because of unpacked non-POD
field 'INNER OUTER::inner'
Abort(coredump)

With the latest compiler on IA64, we try to pack the structure.  This results
in no warning messages but we then generate a Bus error, probably due to
illegal alignment.

So the test never worked but this still seems like a regression in behaviour.

The HP compiler seems to handle this, it compiles and runs without calling
abort.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42837



[Bug middle-end/42837] [4.5 Regression] FAIL: g++.dg/abi/packed1.C execution test

2010-02-17 Thread mmitchel at gcc dot gnu dot org


--- Comment #5 from mmitchel at gcc dot gnu dot org  2010-02-17 17:27 
---
Do we have any evidence that this is actually a regression?  If this test has
never worked on IA64/HPPA, then this is not a regression.  It should then be
XFAIL'd on those platforms, with this PR left open.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42837



[Bug middle-end/42837] [4.5 Regression] FAIL: g++.dg/abi/packed1.C execution test

2010-02-16 Thread sje at cup dot hp dot com


--- Comment #4 from sje at cup dot hp dot com  2010-02-16 18:50 ---
Forgot to mention that the test was added along with a fix for PR c++/41788.
I don't think the fix or the test are bad though.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42837



[Bug middle-end/42837] [4.5 Regression] FAIL: g++.dg/abi/packed1.C execution test

2010-02-16 Thread sje at cup dot hp dot com


--- Comment #3 from sje at cup dot hp dot com  2010-02-16 18:48 ---
I would guess this failure started with r156088 when the test was added.  I
think the test is OK but that ia64 can't handle packing OUTER when one of the
fields in OUTER is a struct with a virtual function.


-- 

sje at cup dot hp dot com changed:

   What|Removed |Added

 CC||sje at cup dot hp dot com
 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-02-16 18:48:29
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42837



[Bug middle-end/42837] [4.5 Regression] FAIL: g++.dg/abi/packed1.C execution test

2010-01-24 Thread danglin at gcc dot gnu dot org


--- Comment #2 from danglin at gcc dot gnu dot org  2010-01-24 14:59 ---
The failure also occurs on hppa2.0w-hp-hpux11.11.


-- 

danglin at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||danglin at gcc dot gnu dot
   ||org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42837



[Bug middle-end/42837] [4.5 Regression] FAIL: g++.dg/abi/packed1.C execution test

2010-01-22 Thread rguenth at gcc dot gnu dot org


--- Comment #1 from rguenth at gcc dot gnu dot org  2010-01-22 11:20 ---
*** Bug 42838 has been marked as a duplicate of this bug. ***


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42837



[Bug middle-end/42837] [4.5 Regression] FAIL: g++.dg/abi/packed1.C execution test

2010-01-22 Thread rguenth at gcc dot gnu dot org


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

   Target Milestone|--- |4.5.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42837