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

George Burgess IV via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 10:34:37 PST 2017


george.burgess.iv added a comment.

Thanks for working on this!



================
Comment at: lib/CodeGen/StackProtector.cpp:485
+
+  std::string ReasonStr;
+  switch (Reason())
----------------
Nit: Can we use a `StringRef` here instead?


https://reviews.llvm.org/D29023





More information about the llvm-commits mailing list