[PATCH] D29517: This patch adds a predicateinfo intrinsic, as part of splitting up D29316.

[Daniel Berlin via Phabricator via llvm-commits]

dberlin added inline comments.


================
Comment at: docs/LangRef.rst:12689
+
+'``llvm.predicateinfo``' Intrinsic
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
----------------
sanjoy wrote:
> How about calling this `llvm.ssa_copy` or something like that, given how general this is?
I kind of deliberately gave it a name that made it so people wouldn't try to use it as a generic copy intrinsic, because i'm not sure we want that used elsewhere :)

But if we do, ...


================
Comment at: docs/LangRef.rst:12702
+
+The first argument is an operand to which predicate info is attached.
+
----------------
sanjoy wrote:
> "attached" is kind of vague -- can you be more specific here?
Sure, would "the first argument is an operand which the PredicateInfo utility creates information about.  Transforms/Utils/PredicateInfo.h has more information, but a brief overview is below".

work?

IE how specific do you want it?



https://reviews.llvm.org/D29517