[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