[PATCH] D19372: NFC. Move Verifier::verifyIntrinsicType to Intrinsics.h

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Thu May 5 17:41:51 PDT 2016


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM w/comment addressed.


================
Comment at: include/llvm/IR/Intrinsics.h:137
@@ -136,1 +136,3 @@
 
+  /// Verify that the specified type (which comes from an intrinsic argument or
+  /// return value) matches the type constraints specified by the .td file (e.g.
----------------
I know your just moving code, but would you mind fixing this comment as well?  It's really not clear that the function is actively modifying the ArrayRef passed in.


http://reviews.llvm.org/D19372





More information about the llvm-commits mailing list