[PATCH] D143620: [RISCV] Use OS-specific SafeStack ABI for Fuchsia

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 10 07:46:20 PST 2023


asb added a comment.

Thanks @mcgrathr - I've left an inline query.



================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:14249
+
+  return TargetLowering::getIRStackGuard(IRB);
+}
----------------
I've not looked at the safe stack pointer code before, but I'd expect this to call `TargetLowering::getSafeStackPointerLocation(IRB)`. Why doesn't it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143620



More information about the llvm-commits mailing list