[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