[PATCH] D69723: [RISCV] Fix wrong CFI directives

Shiva Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 11 21:32:49 PST 2019


shiva0217 added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVFrameLowering.cpp:273
 
     // Emit ".cfi_def_cfa_offset FirstSPAdjustAmount" if using an sp-based CFA
     if (!hasFP(MF)) {
----------------
Should we remove this cfi directive, too?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69723





More information about the llvm-commits mailing list