[PATCH] D89645: [InferAttrs] Add nocapture/writeonly to string/mem libcalls

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 21 14:27:20 PDT 2020


fhahn requested changes to this revision.
fhahn added a comment.
This revision now requires changes to proceed.

In D89645#2345674 <https://reviews.llvm.org/D89645#2345674>, @efriedma wrote:

> Almost all the new setDoesNotCapture calls are wrong.  If a call returns a pointer base on an argument, it can't be marked nocapture.

Oh right, I was only thinking about stores & co for `nocapture`. Thanks!


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

https://reviews.llvm.org/D89645



More information about the llvm-commits mailing list