https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78516

--- Comment #17 from joseph at codesourcery dot com <joseph at codesourcery dot 
com> ---
On Tue, 20 Dec 2016, bergner at gcc dot gnu.org wrote:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78516
> 
> --- Comment #16 from Peter Bergner <bergner at gcc dot gnu.org> ---
> (In reply to Vladimir Makarov from comment #15)
> > Sorry, I applied your changes manually and did a typo.  The line
> > 
> > SET_SRC (curr_insn_set) = new_reg;
> > 
> > should be removed.
> > 
> > I tested this patch with the typo on x86-64 and no new failures occurred. 
> > Strange.
> 
> That would suggest on x86-64, SET_SRC (curr_insn_set) is never a subreg for 
> the
> insns that we process here.  As they say, better to be lucky than good! :-) 
> With the SPE, we're just not lucky as shown by Joseph's last run.  I'm sure it
> will pass after removing the unwanted line above.

With this fix, my build completes OK and the number of FAILs is down from 
105 to 62 in g++.sum and from 246 to 188 in gcc.sum (again, this is a 
rough comparison with a build from a month ago rather than an exact 
comparison against unmodified current mainline).

Reply via email to