This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices at dwf_regno
- From: "Ulrich Weigand" <uweigand at de dot ibm dot com>
- To: macro at codesourcery dot com (Maciej W. Rozycki)
- Cc: rohitarulraj at freescale dot com, edmar at freescale dot com (Edmar Wienskoski), dje dot gcc at gmail dot com (David Edelsohn), gcc-patches at gcc dot gnu dot org (gcc-patches at gcc dot gnu dot org), amodra at gmail dot com (Alan Modra), jakub at redhat dot com (Jakub Jelinek)
- Date: Mon, 13 Oct 2014 15:20:25 +0200 (CEST)
- Subject: Re: [RFC: Patch, PR 60102] [4.9/4.10 Regression] powerpc fp-bit ices at dwf_regno
- Authentication-results: sourceware.org; auth=none
Maciej W. Rozycki wrote:
> On Thu, 9 Oct 2014, Maciej W. Rozycki wrote:
>
> > Seeing Rohit got good results it has struck me that perhaps one of the
> > patches I had previously reverted, to be able to compile GCC in the first
> > place, interfered with this fix -- I backed out all the subsequent patches
> > to test yours and Rohit's by themselves only. And it was actually the
> > case, with this change:
> >
> > 2013-05-21 Christian Bruel <christian.bruel@st.com>
> >
> > * dwarf2out.c (multiple_reg_loc_descriptor): Use dbx_reg_number for
> > spanning registers. LEAF_REG_REMAP is supported only for contiguous
> > registers. Set register size out of the PARALLEL loop.
> >
> > back in place, in addition to your fix, I get an all-passed score for
> > gdb.base/store.exp. So your change looks good and my decision to back out
> > the other patches unfortunate. I'll yet run full e500v2 testing now to
> > double check, and let you know what the results are, within a couple of
> > hours if things work well.
>
> It took a bit more because I saw some regressions that I wanted to
> investigate. In the end they turned out intermittent and the failures
> happen sometimes whether your change is applied or not. So I'm fine with
> your change, thanks for your work and patience.
Thanks for verifying!
David, is the patch OK to commit now?
Bye,
Ulrich
--
Dr. Ulrich Weigand
GNU/Linux compilers and toolchain
Ulrich.Weigand@de.ibm.com