craig.topper added inline comments.

================
Comment at: llvm/lib/Target/RISCV/RISCVFrameLowering.cpp:58
 
   const auto *RVFI = MF.getInfo<RISCVMachineFunctionInfo>();
   if (RVFI->useSaveRestoreLibCalls(MF)) {
----------------
Can you add a FIXME here? Using x27 should hopefully remove this restriction


================
Comment at: llvm/lib/TargetParser/RISCVTargetParser.cpp:105
+bool isX27ReservedByDefault(const Triple &TT) {
   // X18 is reserved for the ShadowCallStack ABI (even when not enabled).
   return TT.isOSFuchsia() || TT.isAndroid();
----------------
X18->X27


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146463

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to