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

Luís Marques via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 12 07:31:54 PST 2019


luismarques marked 2 inline comments as done.
luismarques 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)) {
----------------
shiva0217 wrote:
> Should we remove this cfi directive, too?
Yup, that was a silly oversight. Thanks!


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