[PATCH] D129516: [Intrinsics] Add `nocallback` to the memset/cpy/move intrinsics

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/include/llvm/IR/Intrinsics.td:622
                             [IntrArgMemOnly, IntrWillReturn, IntrNoFree,
+                             IntrNoCallback,
                              NoCapture<ArgIndex<0>>, NoCapture<ArgIndex<1>>,
----------------
nikic wrote:
> Why can't we use DefaultAttrsIntrinsic here? llvm.memcpy can't synchronize, right?
The problem is the volatile flag. I don't think we can/should make them nosync as long as we have that one...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129516