[PATCH] D31950: SamplePGO: convert callsite samples map key from callsite_location to callsite_location+callee_name

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 10:05:57 PDT 2017


davidxl added inline comments.


================
Comment at: include/llvm/ProfileData/SampleProf.h:281
+      return &FS->getValue();
+    // If we cannot find exact match of the callee name, return the FS with
+    // the max total count.
----------------
Is it correct to return the max count samples? What is the reason this is needed?


https://reviews.llvm.org/D31950





More information about the llvm-commits mailing list