On Mon, Oct 13, 2014 at 9:20 AM, Ulrich Weigand <uweig...@de.ibm.com> wrote: > 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.br...@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?
Okay with me. Thanks! David
