[PATCH] D147740: [llvm-profdata] Refactoring Sample Profile Reader to increase FDO build speed using MD5 as key to Sample Profile map

[William Junda Huang via Phabricator via llvm-commits]

huangjd added a comment.

To add



================
Comment at: llvm/include/llvm/ProfileData/SampleProfReader.h:415
+    uint64_t MD5 = MD5Hash(CanonName);
+    auto Ret = Profiles.try_emplace(MD5, FunctionSamples());
+    if (!Ret.second) {
----------------
davidxl wrote:
> Why not using the new interface SampleProfileMapTryEmplace?
This is one of the exception case because of the additional processing at line 425.  This function intends to succeed even if a collision happens


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147740/new/

https://reviews.llvm.org/D147740