On Tue, Mar 27, 2018 at 11:04:35AM +0000, Kumar, Venkataramanan wrote:
> > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk if it
> > helps for SPEC? Venkat, do you think you could benchmark it in the setup
> > where you've measured the slowdown to see if it helps? I see the patch
> > changes the loop:
Thanks for benchmarking it.
> The patch causes regression benchmark when I measured on my Ryzen box (>4%)
> .
>
> GCC trunk: -O3 -maxv2 -mprefer-avx128 -O3 -march=znver1
> 429.mcf 9120 238 38.3 * 9120 227 40.2 *
>
> GCC patch:
> 429.mcf 9120 251 36.3 * 9120 236 38.6 *
So, has 429.mcf improved then compared to 7.x sufficiently that we can turn
PR78200 into just [7 Regression], without adding any patch?
Jakub
