[PATCH] D29027: [Stack Protection] Add remark for reasons why Stack Protection has been applied

[James Henderson via Phabricator via cfe-commits]

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

Thanks for the comments. I have updated the diff accordingly, by removing the unknown case, adding a check to distinguish between the function attribute and command-line switch, and expanded the alloca remark to include VLAs. I also replaced the reference to SSP from the switch and remark text as suggested. To go with that, I changed the DiagGroup name.


https://reviews.llvm.org/D29027

Files:
  include/clang/Basic/DiagnosticFrontendKinds.td
  include/clang/Basic/DiagnosticGroups.td
  lib/CodeGen/CodeGenAction.cpp
  test/CodeGen/stack-protector-remarks.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29027.85570.patch
Type: text/x-patch
Size: 5497 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170124/754dd8a1/attachment-0001.bin>