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

--- Comment #2 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by John David Anglin <dang...@gcc.gnu.org>:

https://gcc.gnu.org/g:1ea45291af0bc8f7b6dff67a0f23be662b2f9908

commit r15-5049-g1ea45291af0bc8f7b6dff67a0f23be662b2f9908
Author: John David Anglin <dang...@gcc.gnu.org>
Date:   Fri Nov 8 16:34:41 2024 -0500

    hppa: Fix handling of secondary reloads involving a SUBREG

    This is fairly subtle.

    When handling spills for SUBREG arguments in pa_emit_move_sequence,
    alter_subreg may be called.  It in turn calls adjust_address_1 and
    change_address_1.  change_address_1 calls pa_legitimate_address_p
    to validate the new spill address.  change_address_1 generates an
    internal compiler error if the address is not valid.  We need to
    allow 14-bit displacements for all modes when reload_in_progress
    is true and strict is false to prevent the internal compiler error.

    SUBREGs are only used with the general registers, so the spill
    should result in an integer access.  14-bit displacements are okay
    for integer loads and stores but not for floating-point loads and
    stores.

    Potentially, the change could break the handling of spills for the
    floating point-registers but I believe these are handled separately
    in pa_emit_move_sequence.

    This change fixes the build of symmetrica-3.0.1+ds.

    2024-11-08  John David Anglin  <dang...@gcc.gnu.org>

    gcc/ChangeLog:

            PR target/117443
            * config/pa/pa.cc (pa_legitimate_address_p): Allow any
            14-bit displacement when reload is in progress and strict
            is false.

Reply via email to