[PATCH] D62190: [RISCV] Allow shrink wrapping for RISC-V

Luís Marques via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 16 15:46:45 PDT 2019


luismarques requested changes to this revision.
luismarques added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/Target/RISCV/RISCVFrameLowering.cpp:181
+  if (!MBBI->isTerminator())
+    MBBI = std::next(MBBI);
+
----------------
Document why we advance exactly one instruction.


================
Comment at: lib/Target/RISCV/RISCVFrameLowering.cpp:546
+
+  if (!RVFI->getUseSaveRestoreLibCalls())
+    return true;
----------------
This patch is using this function but it seems like D62686 no longer introduces it.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62190





More information about the llvm-commits mailing list