[PATCH] Fix PR62077

Jason Merrill jason@redhat.com
Wed Aug 13 17:50:00 GMT 2014


On 08/13/2014 10:28 AM, Richard Biener wrote:
> Sofar the patch survived building stage2 in a LTO bootstrap on the
> 4.9 branch, full testing is scheduled for trunk.

The patch breaks a lot of C++ testcases, such as
g++.old-deja/g++.other/cvt1.C; I think you need to share the "set the 
canonical type" code with the template path.

> Jason, are you happy with that (esp. ripping out the odd
> type completion stuff that also messes with types recorded in
> said hashtable)?

I'm nervous about it, since it leads to ARRAY_TYPEs with different 
TYPE_ALIGN than their elements, though I'm not sure this actually breaks 
anything.  Perhaps we could copy TYPE_ALIGN and TYPE_USER_ALIGN at the 
same place we copy TYPE_NEEDS_CONSTRUCTING.

Jason



More information about the Gcc-patches mailing list