[PATCH] D21121: IR: Introduce llvm.type.checked.load intrinsic.

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.
Herald added subscribers: nlopes, ormris, jdoerfert, Prazek.
Herald added projects: LLVM, All.


================
Comment at: llvm/trunk/include/llvm/IR/Intrinsics.td:673
+                                      [llvm_ptr_ty, llvm_i32_ty, llvm_metadata_ty],
+                                      [IntrNoMem]>;
+
----------------
Why is this intrinsic marked as IntrNoMem? LangRef says it's supposed to be ArgMemOnly + ReadOnly, and that's what I'd normally expect from a load-like intrinsic...


Repository:
  rL LLVM

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

https://reviews.llvm.org/D21121