[PATCH] D31317: Change the default attributes for llvm.prefetch to enable optimization.

[Sanjoy Das via Phabricator via llvm-commits]

sanjoy added inline comments.


================
Comment at: include/llvm/IR/Intrinsics.td:334
 
-// IntrArgMemOnly is more pessimistic than strictly necessary for prefetch,
-// however it does conveniently prevent the prefetch from being reordered
-// with respect to nearby accesses to the same memory.
-def int_prefetch      : Intrinsic<[],
-                                  [llvm_ptr_ty, llvm_i32_ty, llvm_i32_ty,
-                                   llvm_i32_ty],
-                                  [IntrArgMemOnly, NoCapture<0>]>;
+// IntrArgMemOrArgMemOnly is a little more pessimistic than strictly necessary
+// for prefetch, however it does conveniently prevent the prefetch from being
----------------
Should the comment read `IntrInaccessibleMemOrArgMemOnly is a ...` ?


https://reviews.llvm.org/D31317