[PATCH] D44077: Clear the stack protector after checking it

Sat May 18 05:33:30 PDT 2019

Flakebi updated this revision to Diff 200146.
Flakebi added a comment.
Herald added subscribers: mstorsjo, hiraditya.
Herald added a project: LLVM.

The ssp is now set to zero in the SDAG variant and when a stack guard check function is used. I am not familiar with the selection DAG so please correct me if it can be improved.
I also added a test as requested.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D44077

Files:
  llvm/lib/CodeGen/SafeStack.cpp
  llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
  llvm/lib/CodeGen/StackProtector.cpp
  llvm/test/CodeGen/X86/stack-protector-zero.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44077.200146.patch
Type: text/x-patch
Size: 8833 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190518/f0f7136c/attachment.bin>