aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/CodeGen/CodeGen.cpp
diff options
context:
space:
mode:
authorMatt Arsenault <Matthew.Arsenault@amd.com>2025-03-04 10:04:14 +0700
committerGitHub <noreply@github.com>2025-03-04 10:04:14 +0700
commit8476a5d480304bf7bd934c660a159e1c6906a69d (patch)
treed59402054273ba2599da91acb0ec637a982dc6de /llvm/lib/CodeGen/CodeGen.cpp
parentce1a18e2c714f39fe72cd46aa04faed29ad23cb6 (diff)
downloadllvm-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