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

--- Comment #22 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Richard Sandiford
<rsand...@gcc.gnu.org>:

https://gcc.gnu.org/g:66d01cc3f4a248ccc471a978f0bfe3615c3f3a30

commit r11-11467-g66d01cc3f4a248ccc471a978f0bfe3615c3f3a30
Author: Richard Sandiford <richard.sandif...@arm.com>
Date:   Tue Jun 4 13:47:35 2024 +0100

    rtl-ssa: Extend m_num_defs to a full unsigned int [PR108086]

    insn_info tried to save space by storing the number of
    definitions in a 16-bit bitfield.  The justification was:

      // ...  FIRST_PSEUDO_REGISTER + 1
      // is the maximum number of accesses to hard registers and memory, and
      // MAX_RECOG_OPERANDS is the maximum number of pseudos that can be
      // defined by an instruction, so the number of definitions should fit
      // easily in 16 bits.

    But while that reasoning holds (I think) for real instructions,
    it doesn't hold for artificial instructions.  I don't think there's
    any sensible higher limit we can use, so this patch goes for a full
    unsigned int.

    gcc/
            PR rtl-optimization/108086
            * rtl-ssa/insns.h (insn_info): Make m_num_defs a full unsigned int.
            Adjust size-related commentary accordingly.

    (cherry picked from commit cd41085a37b8288dbdfe0f81027ce04b978578f1)
  • [Bug rtl-optimization/108086] [... cvs-commit at gcc dot gnu.org via Gcc-bugs

Reply via email to