[PATCH] D44399: [ThinLTO] Add funtions in callees metadata to CallGraphEdges

Taewook Oh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 12 16:07:32 PDT 2018


twoh added a comment.

@tejohnson Thanks for the clarification. Regarding hotness, I'm not sure if providing "some" hotness is better than leaving it as unknown if profile data is not provided (If profile data is given, as you said, VP metadata will be attached to the callsite). I'm afraid that synthesized hotness may confuse optimizers, but please let me know if you have different idea.

I'll update the test to check `f2` as well. Thanks for the comments!


Repository:
  rL LLVM

https://reviews.llvm.org/D44399





More information about the llvm-commits mailing list