On 19 October 2016 at 10:34, Kyrill Tkachov <kyrylo.tkac...@foss.arm.com> wrote: > > On 19/10/16 07:55, Christophe Lyon wrote: >> >> On 18 October 2016 at 17:35, Christophe Lyon <christophe.l...@linaro.org> >> wrote: >>> >>> On 18 October 2016 at 16:45, Bernd Edlinger <bernd.edlin...@hotmail.de> >>> wrote: >>>> >>>> On 10/18/16 10:36, Christophe Lyon wrote: >>>>> >>>>> I am seeing a lot of regressions since this patch was committed: >>>>> >>>>> http://people.linaro.org/~christophe.lyon/cross-validation/gcc/trunk/241273/report-build-info.html >>>>> >>>>> (you can click on "REGRESSED" to see the list of regressions, "sum" >>>>> and "log" to download >>>>> the corresponding .sum/.log) >>>>> >>>>> Thanks, >>>>> >>>>> Christophe >>>>> >>>> Oh, sorry, I have completely missed that. >>>> Unfortunately I have tested one of the good combinations. >>>> >>>> I have not considered the case that in and out could be >>>> the same register. But that happens here. >>>> >>>> >>>> This should solve it. >>>> >>>> Can you give it a try? >>>> >>> Sure, I have started the validations, it will take a few hours. >>> >> It looks OK, thanks. > > > Ok. Thanks for testing Christophe. > Sorry for not catching it in review. > Kyrill >
Since it was painful to check that this 2nd patch fixed all the regressions observed in all the configurations, I ran another validation with the 2 patches combined, on top of r241272 to check the effect of the two over the previous reference. It turns out there is still a failure: http://people.linaro.org/~christophe.lyon/cross-validation/gcc-test-patches/241272-r241273-combined/report-build-info.html gcc.c-torture/execute/pr34971.c -O0 execution test now fails on arm-none-linux-gnueabihf when using thumb mode, expect when targeting cortex-a5. Christophe >>>> >>>> Thanks >>>> Bernd. > >
