[PATCH] D31952: Build SymbolMap in SampleProfileLoader to help matchin function names with suffix.

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: lib/Transforms/IPO/SampleProfile.cpp:1439
+      if (!r.second)
+        SymbolMap.erase(r.first);
+    }
----------------
danielcdh wrote:
> davidxl wrote:
> > Why erasing the existing entry?
> Updated the comment to explain why we remove the existing entry.
but the new entry is not inserted either here. Should it just reset r.second = F? Also In presence of conflict, how do we know which one to pick?


https://reviews.llvm.org/D31952