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

Reply via email to