[PATCH] D100919: [AArch64] Support customizing stack protector guard

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 17 11:40:08 PDT 2021


nickdesaulniers marked 3 inline comments as done.
nickdesaulniers added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:1954
+      // than 23760.
+      // It might be nice to use AArch64::MOVi32imm here, which would get
+      // expanded in PreSched2 after PostRA, but our lone scratch Reg already
----------------
dmgreen wrote:
> It may be possible to do something earlier, where a we add the MOVi32imm earlier before registry allocation and use that as the register for the ldr offset. That would keep it simpler than trying to find scratch registers, and may be optimized better by the pipeline.
Sure; added that as a comment; hopefully we wont need it, but "never say never (again)."


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100919



More information about the llvm-commits mailing list