[PATCH] D60404: Improve hashing for time profiler

Anton Afanasyev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 8 07:33:09 PDT 2019


anton-afanasyev added inline comments.


================
Comment at: llvm/lib/Support/TimeProfiler.cpp:124
+    for (const auto &E : CountAndTotalPerName) {
+      SortedTotals.push_back(std::pair<std::string, CountAndDurationType>(
+          E.getKey(), E.getValue()));
----------------
takuto.ikuta wrote:
> use emplace_back here?
Yes, thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60404





More information about the llvm-commits mailing list