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

--- Comment #4 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Tamar Christina <[email protected]>:

https://gcc.gnu.org/g:d956c5ad043f767bcde25e09ec48917afb2c8e8c

commit r17-4248-gd956c5ad043f767bcde25e09ec48917afb2c8e8c
Author: Tamar Christina <[email protected]>
Date:   Tue Sep 15 08:10:40 2026 +0100

    AArch64: use alternate fix for FFR CSE bug [PR126629]

    In the review for the fix for PR126629 Alfie pointed out that this would
break
    FFR widening loads.  At the time I thought I could fix REE to handle this,
but
    the patch didn't look appropriate.

    So after revisiting Alfie's original idea, my main issue was that it would
be
    had to maintain.

    This approach instead moves the entire SET src UNSPEC into the update_ffr
    "token" parameter.  This allows us to fix the CSE issue, while keeping
    maintainability and supporting FFR widening loads from the vectorizer.

    gcc/ChangeLog:

            PR target/126629
            * config/aarch64/aarch64-sve-builtins-base.cc
            (emit_ldf1_ffr_update): New.
            (svldff1_gather_impl, svldff1_gather_extend, svldxf1_impl,
            svldxf1_extend_impl): Use it.
            * config/aarch64/aarch64-sve.md (aarch64_update_ffr<mode>): Update
insn.

Reply via email to