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? Bye, Ulrich -- Dr. Ulrich Weigand GNU/Linux compilers and toolchain ulrich.weig...@de.ibm.com
