[PATCH] D87956: [IR] add fn attr for no_stack_protector; prevent inlining on mismatch

Nick Desaulniers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 22 16:43:42 PDT 2020


nickdesaulniers added a comment.

In D87956#2348446 <https://reviews.llvm.org/D87956#2348446>, @void wrote:

> Why not just use "expected-error" and the like?

Ah, brilliant! Thank you!  Will wait to submit until tomorrow morning.



================
Comment at: llvm/docs/BitCodeFormat.rst:1072
+* code 69: ``byref``
+* code 70: ``mustprogress``
+* code 71: ``nossp``
----------------
void wrote:
> I'm tempted to say that these two adds should be a separate commit, but it's a nit and I'll leave it up to you.
Done in rG4abaf0ec0a3c


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87956



More information about the cfe-commits mailing list