diff options
author | Matt Arsenault <Matthew.Arsenault@amd.com> | 2025-03-04 10:04:14 +0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-03-04 10:04:14 +0700 |
commit | 8476a5d480304bf7bd934c660a159e1c6906a69d (patch) | |
tree | d59402054273ba2599da91acb0ec637a982dc6de /llvm/lib/CodeGen/CodeGen.cpp | |
parent | ce1a18e2c714f39fe72cd46aa04faed29ad23cb6 (diff) | |
download | llvm-8476a5d480304bf7bd934c660a159e1c6906a69d.zip llvm-8476a5d480304bf7bd934c660a159e1c6906a69d.tar.gz llvm-8476a5d480304bf7bd934c660a159e1c6906a69d.tar.bz2 |
SplitKit: Fix rematerialization undoing subclass based split (#122110)
This fixes an allocation failure in the new test.
In cases where getLargestLegalSuperClass can inflate the register class,
rematerialization could effectively undo a split which was done to
inflate
the register class, if the defining instruction can only write a
subclass
and the use can read the superclass.
Some of the x86 tests changes look like improvements, but some are
likely regressions.
I'm not entirely sure this is the correct place to fix this. It also
seems more complicated than necessary, but the decision to change
the register class is far removed from the point where the decision
to split the virtual register is made. I'm also also not sure if this
should be considering the register classes of all the use indexes
in getUseSlots, rather than just checking if this use index instruction
reads the register.
Diffstat (limited to 'llvm/lib/CodeGen/CodeGen.cpp')
0 files changed, 0 insertions, 0 deletions