[PATCH] D48105: [llvm][Instrumentation] Add Call Graph Profile pass

[Michael Spencer via Phabricator via llvm-commits]

Bigcheese added inline comments.


================
Comment at: llvm/trunk/lib/Transforms/Instrumentation/CGProfile.cpp:68
+
+        uint64_t &Count = Counts[std::make_pair(&F, CalledF)];
+        Count = SaturatingAdd(Count, *BBCount);
----------------
davidxl wrote:
> You may also check indirect callsite to see if it has value profile data (indirect call target). If it has, you can create edges to the targets as well.
> 
> (I assume this information is for function layout purpose by the linker).
Is there an example where this is used?  I'm not familiar with value profile data.


https://reviews.llvm.org/D48105