On 6 November 2017 at 14:05, Wilco Dijkstra <wilco.dijks...@arm.com> wrote: > Maxim Kuvyrkov wrote: >> > On Nov 2, 2017, at 9:41 PM, Wilco Dijkstra <wilco.dijks...@arm.com> wrote: >> > >> > The Arm backend sets the default sched-pressure algorithm to >> > SCHED_PRESSURE_MODEL. Benchmarking on AArch64 shows this >> > speeds up floating point performance on SPEC - eg. CactusBSSN improves >> > by ~16%. The gains are mostly due to less spilling, so enable this on >> > AArch64 >> > by default. >> >> Hi Wilco, >> >> Any notable regressions? > > No, nothing that stands out. There were a few regressions on Cortex-A57 but > none > reproduced on Cortex-A72, so they are not real regressions. The gains do > reproduce > and instruction counts are lower (most binaries show a significant reduction > in spills). > > Wilco
Hi Wilco, After you committed this patch (r254378), I noticed a few regressions: FAIL: gcc.target/aarch64/subs_compare_1.c scan-assembler-not cmp\\tw[0-9]+, w[0-9]+ FAIL: gcc.target/aarch64/subs_compare_1.c scan-assembler-times subs\\tw[0-9]+, w[0-9]+, w[0-9]+ 1 (found 0 times) FAIL: gcc.target/aarch64/subs_compare_2.c scan-assembler-times subs\\tw[0-9]+, w[0-9]+, #4 1 (found 0 times) I'm still catching-up, so maybe you already fixed this in a subsequent commit? (It's still failing in my most recent validation to date, r254467). Thanks, Christophe