[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