On Wed, 10 Jul 2024, Uros Bizjak wrote:

> emit_store_flag_1 calculates scode (swapped condition code) at the
> beginning of the function from the value of code variable.  However,
> code variable may change before scode usage site, resulting in
> invalid stalled scode value.
> 
> Move calculation of scode value just before its only usage site to
> avoid stalled scode value.
> 
>     PR middle-end/115836
> 
> gcc/ChangeLog:
> 
>     * expmed.cc (emit_store_flag_1): Move calculation of
>     scode just before its only usage site.
> 
> Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}.
> 
> Also tested with original and minimized preprocessed source.
> Unfortunately, even with the minimized source, the compilation takes
> ~5 minutes, and IMO such a trivial fix does not warrant that high
> resource consumption.
> 
> OK for master and release branches?

OK.

Thanks,
Richard.

Reply via email to