[PATCH] D68979: [RISCV] Handle variable sized objects with the stack need to be realigned

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 07:12:25 PDT 2019


lenary added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/stack-realignment-with-variable-sized-objects.ll:33
+; RV32I-NEXT:    lw s0, 120(sp)
+; RV32I-NEXT:    .cfi_def_cfa sp, 128
+; RV32I-NEXT:    lw ra, 124(sp)
----------------
This doesn't look right. Why does this directive come after the restore of `s0` and `s1` (but not `ra`)? Shouldn't it come immediately after the `addi` instruction?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68979/new/

https://reviews.llvm.org/D68979





More information about the llvm-commits mailing list