[PATCH] D39480: Include GUIDs from the same module when computing GUIDs that needs to be imported.

Dehao Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 1 11:15:19 PDT 2017


danielcdh marked 2 inline comments as done.
danielcdh added a comment.

> If the indirect calls along this chain were promoted and then inlined into foo in the profiled binary, we cannot do the promotion/inlining of the call back to A.c:baz() during the compile step since it is intra-module, but we need to be able to annotate the call somewhere. And apparently we can't do this on the VP metadata for the indirect call in B.c:bar() since it was inlined into foo() in the profiled binary (Dehao - is that correct?).

That's correct


https://reviews.llvm.org/D39480





More information about the llvm-commits mailing list