Re: [PATCH] Fix PR62077

2014-08-15 Thread Richard Biener
On August 15, 2014 7:25:55 PM CEST, Jason Merrill wrote: >On 08/14/2014 12:28 PM, Jason Merrill wrote: >> On 08/14/2014 05:07 AM, Richard Biener wrote: >>> So - can you take over this C++ frontend issue? >> >> OK. > >Here's what I'm applying to trunk: Thanks Jason. Richard.

Re: [PATCH] Fix PR62077

2014-08-15 Thread Jason Merrill
On 08/14/2014 12:28 PM, Jason Merrill wrote: On 08/14/2014 05:07 AM, Richard Biener wrote: So - can you take over this C++ frontend issue? OK. Here's what I'm applying to trunk: commit 76cce1d62861b6c99e3ecd97bcd607cd242d1efa Author: Jason Merrill Date: Wed Aug 13 13:11:09 2014 -0400

Re: [PATCH] Fix PR62077

2014-08-14 Thread Jason Merrill
On 08/14/2014 05:07 AM, Richard Biener wrote: So - can you take over this C++ frontend issue? OK. Jason

Re: [PATCH] Fix PR62077

2014-08-14 Thread Richard Biener
On Thu, 14 Aug 2014, Richard Biener wrote: > On Wed, 13 Aug 2014, Jason Merrill wrote: > > > 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 lo

Re: [PATCH] Fix PR62077

2014-08-14 Thread Richard Biener
On Wed, 13 Aug 2014, Jason Merrill wrote: > 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.

Re: [PATCH] Fix PR62077

2014-08-13 Thread Jason Merrill
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 typ

Re: [PATCH] Fix PR62077

2014-08-13 Thread Richard Biener
On Wed, 13 Aug 2014, Richard Biener wrote: > > The following fixes the LTO bootstrap miscompares on the branches. > They happen because type_hash_canon can end up returning different > types (a main variant or a variant type) dependent on whether > we garbage collected before or not. > > The pat

[PATCH] Fix PR62077

2014-08-13 Thread Richard Biener
The following fixes the LTO bootstrap miscompares on the branches. They happen because type_hash_canon can end up returning different types (a main variant or a variant type) dependent on whether we garbage collected before or not. The patch makes us never return a variant type from type_hash_can