[PATCH] D29517: This patch adds a predicateinfo intrinsic, as part of splitting up D29316.
Daniel Berlin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 3 21:06:39 PST 2017
dberlin added inline comments.
Comment at: docs/LangRef.rst:12689
> 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.
> "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".
IE how specific do you want it?
More information about the llvm-commits