[PATCH] D138502: [RISCV] Simplify eliminateFrameIndex in advance of reuse [nfc-ish]
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 22 08:12:48 PST 2022
reames added inline comments.
================
Comment at: llvm/test/CodeGen/RISCV/rvv/emergency-slot.mir:114
; CHECK-NEXT: renamable $x23 = ADDI $x2, 1920
- ; CHECK-NEXT: $x24 = LUI 1
- ; CHECK-NEXT: $x24 = ADDIW killed $x24, -2048
- ; CHECK-NEXT: $x24 = ADD $x2, killed $x24
+ ; CHECK-NEXT: SD killed $x1, $x2, 8 :: (store (s64) into %stack.15)
+ ; CHECK-NEXT: SD killed $x5, $x2, 0 :: (store (s64) into %stack.16)
----------------
Note these are spills to the stack. The register allocator has picked a different register to use for the sequence below, and is thus save/restoring a different value here. That's correct, but confusing on first glance.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138502/new/
https://reviews.llvm.org/D138502
More information about the llvm-commits
mailing list