diff options
author | Richard Sandiford <richard.sandiford@arm.com> | 2023-02-02 14:53:34 +0000 |
---|---|---|
committer | Richard Sandiford <richard.sandiford@arm.com> | 2023-02-02 14:53:34 +0000 |
commit | cd41085a37b8288dbdfe0f81027ce04b978578f1 (patch) | |
tree | 5058c4bfdb1d448a45b7c0f84a06aaca79423a74 /libgcc | |
parent | f4e1b46618ef3bd7933992ab79f663ab9112bb80 (diff) | |
download | gcc-cd41085a37b8288dbdfe0f81027ce04b978578f1.zip gcc-cd41085a37b8288dbdfe0f81027ce04b978578f1.tar.gz gcc-cd41085a37b8288dbdfe0f81027ce04b978578f1.tar.bz2 |
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.
Diffstat (limited to 'libgcc')
0 files changed, 0 insertions, 0 deletions