[PATCH] D37354: Add !targets metadata

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

I don't like the term "targets", in part because that term is generally used to refer to the target architecture. How about "callees"?



================
Comment at: docs/LangRef.rst:4883
+known set of functions that a call or invoke instruction could possibly target
+at run-time. If ``targets`` metadata is attached to a given instruction, it is
+known that the instruction will only target one or more of the functions in the
----------------
run-time -> runtime


================
Comment at: docs/LangRef.rst:4883
+known set of functions that a call or invoke instruction could possibly target
+at run-time. If ``targets`` metadata is attached to a given instruction, it is
+known that the instruction will only target one or more of the functions in the
----------------
hfinkel wrote:
> run-time -> runtime
If ``targets`` metadata is attached to a call site, and any callee is not among the set of functions provided by the metadata, the behavior is undefined.


https://reviews.llvm.org/D37354