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

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

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

commit r16-3841-gf8b6f1d83ba8fd4186100e1e357ba7eb8124bdb7
Author: Stefan Schulze Frielinghaus <[email protected]>
Date:   Thu Jul 24 12:03:43 2025 +0200

    lra: Stop constraint processing on error [PR121205]

    It looks like we didn't have a test so far reaching this point which
    changed with the new hard register constraint tests.  Bootstrap and
    regtest are still running on x86_64.  If they succeed, ok for mainline?

    -- >8 --

    As noted by Sam in the PR, with checking enabled tests
    gcc.target/i386/asm-hard-reg-{1,2}.c fail with an ICE.  If an error is
    detected in curr_insn_transform(), lra_asm_insn_error() is called and
    deletes the current insn.  However, afterwards processing continues with
    the deleted insn and via lra_process_new_insns() we finally call recog()
    for NOTE_INSN_DELETED which ICEs in case of a checking build.  Thus, in
    case of an error during curr_insn_transform() bail out and stop
    processing.

    gcc/ChangeLog:

            PR rtl-optimization/121205
            * lra-constraints.cc (curr_insn_transform): Stop processing on
            error.

Reply via email to