[PATCH] D19407: [SafeStack] [SSP] Use llvm.stackguard intrinsic.

Marcin Koƛcielnicki via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 07:08:11 PDT 2016


koriakin added a comment.

In http://reviews.llvm.org/D19407#409330, @eugenis wrote:

> Cool!
>
> I wonder if we could get SDAG SSP for free now by updating the following code to look at SafeStack, too:
>  getAnalysis<StackProtector>().shouldEmitSDCheck(*LLVMBB)


I don't quite follow - the canary in SafeStack is a different beast (for one, it's on the other stack), and needs much fewer hacks (eg. we don't need to avoid spilling it).


Repository:
  rL LLVM

http://reviews.llvm.org/D19407





More information about the llvm-commits mailing list