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 ja...@redhat.com Date: Wed Aug 13

Re: [PATCH] Fix PR62077

2014-08-15 Thread Richard Biener
On August 15, 2014 7:25:55 PM CEST, Jason Merrill ja...@redhat.com 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-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.C; I

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 lot of C++

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

[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

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 patch

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