https://gcc.gnu.org/bugzilla/show_bug.cgi?id=126056

Filip Kastl <pheeck at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[17 Regression] 25%         |[17 Regression] 25%
                   |slowdown of 470.lbm on AMD  |slowdown of 470.lbm on AMD
                   |Zen5 and 7% 538.imagick_r   |Zen5 since
                   |on Zen4 since               |r17-1845-g23b6a9a41a5c05
                   |r17-1845-g23b6a9a41a5c05    |

--- Comment #2 from Filip Kastl <pheeck at gcc dot gnu.org> ---
(In reply to Filip Kastl from comment #1)
> Another slowdown at the same commit:
> 
> 7% on SPEC CPU 2017

Oh, nevermind.  Wanted to report this

https://lnt.opensuse.org/db_default/v4/SPEC/graph?plot.0=1108.507.0

but it has meanwhile disappeared.  So only the lbm slowdown remains


But looking at the graph for lbm, I think that one will also disappear.  Might
be some sort of code alignment thing or the register allocator getting
(un)lucky.  lbm is a small benchmark and you can see that the runtime oscilates
betwen two values.  So I guess that your patch simply influenced some minor
compiler decision which the big loop nest in the benchmark magnified into 25%
slowdown.

So maybe let's keep this open in case someone wants to dig into this to speedup
the benchmark.  Otherwise I wouldn't worry about it.  The benchmark runtime
will probably eventually drop to the previous values anyway.

Reply via email to