[PATCH] D22204: Add getReturnedArgOperand to Call/InvokeInst, CallSite

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 10 14:41:30 PDT 2016


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

LGTM, the nit can be optionally addressed.


================
Comment at: include/llvm/IR/Attributes.h:343-344
@@ -343,1 +342,4 @@
+  /// of a parameter with the specified attribute is provided.
+  bool hasAttrSomewhere(Attribute::AttrKind Kind,
+                        unsigned *Index = nullptr) const;
 
----------------
Maybe this interface would be nicer if it returned an `Optional<bool>`.  I think it would have minimal impact on the callers.


http://reviews.llvm.org/D22204





More information about the llvm-commits mailing list