[PATCH] D90702: [SafeStack] Make sure SafeStack does not break musttail call contract

Xun Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 3 11:37:36 PST 2020


lxfind created this revision.
lxfind added reviewers: wenlei, vitalybuka, rnk.
Herald added subscribers: llvm-commits, modimo, hiraditya.
Herald added a project: LLVM.
lxfind requested review of this revision.

SafeStack instrumentation should not insert anything inbetween musttail call and return instruction.
For every ReturnInst that needs to be instrumented, we adjust the insertion point to the musttail call if exists.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D90702

Files:
  llvm/lib/CodeGen/SafeStack.cpp
  llvm/test/Transforms/SafeStack/X86/musttail.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D90702.302642.patch
Type: text/x-patch
Size: 6707 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201103/9aff2577/attachment.bin>


More information about the llvm-commits mailing list