RFA: PATCH to finalize_record_size to avoid ABI changes from -Wpacked (c++/41788)

Jason Merrill jason@redhat.com
Fri Apr 9 15:00:00 GMT 2010


Bug 41788 pointed out that -Wpacked changed the layout of a class by 
unsetting TYPE_PACKED when it seemed unnecessary.  For 4.5 I fixed that 
bug by making it clear that the flag is actually necessary for that 
testcase, but in general it seems wrong for the compiler to change 
layout based on a warning flag.  So this patch changes 
finalize_record_size to only give the warning, not actually unset the flag.

Tested x86_64-unknown-linux-gnu.  OK for trunk?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 41788-warn-packed.patch
Type: text/x-patch
Size: 696 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20100409/68f9c1d9/attachment.bin>


More information about the Gcc-patches mailing list