[PATCH] D27521: AArch64 Cortex-A57 FDIV/FSQRT scheduling fix (W-unit)

[Kristof Beyls via Phabricator via llvm-commits]

kristof.beyls added a comment.

In https://reviews.llvm.org/D27521#627385, @andrew.zhogin wrote:

> In https://reviews.llvm.org/D27521#625100, @rengolin wrote:
>
> > Can you run any benchmark and share relative numbers? Despite the fact that the change is obvious, you may uncover pre-existing bugs in the compiler that need addressing before we can change this. The LLVM test-suite, SPEC, EEMBC, anything would help to have an idea.
>
>
> I don't have access to Cortex-A57 system.
>  I did only check-all runs at my X86-64 desktop (green).


I've run this patch on top of r290135 on test-suite, spec2K, spec2K6 and a few internal benchmarks.
I saw 2 minor regressions and 2 bigger improvements on internal benchmarks.
Across test-suite, spec2K and spec2K6, all results were within noise levels.


https://reviews.llvm.org/D27521