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

Dehao Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 17 14:33:11 PDT 2017


danielcdh marked an inline comment as done.
danielcdh added inline comments.


================
Comment at: lib/Transforms/IPO/SampleProfile.cpp:1439
+      if (!r.second)
+        SymbolMap.erase(r.first);
+    }
----------------
davidxl wrote:
> Why erasing the existing entry?
Updated the comment to explain why we remove the existing entry.


https://reviews.llvm.org/D31952





More information about the llvm-commits mailing list