[PATCH] D29023: [Stack Protection] Add diagnostic information for why stack protection was applied to a function

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 22 06:45:34 PST 2017


jhenderson updated this revision to Diff 89356.
jhenderson edited the summary of this revision.
jhenderson added a comment.

After staring at this for a while, and discussing terminology, I have come to the conclusion that the best way to report these remarks is not via a new diagnostic class, but rather through simply using emitOptimizationRemark. This updated diff simplifies things substantially from the previous version and also gives us clang support for free via -Rpass=stack-protector.


https://reviews.llvm.org/D29023

Files:
  lib/CodeGen/StackProtector.cpp
  test/CodeGen/X86/stack-protector-remarks.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29023.89356.patch
Type: text/x-patch
Size: 6634 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170222/2b5969af/attachment.bin>


More information about the llvm-commits mailing list