[PATCH] D60404: Improve hashing for time profiler

[Anton Afanasyev via Phabricator via llvm-commits]

anton-afanasyev marked 2 inline comments as done.
anton-afanasyev added inline comments.


================
Comment at: llvm/lib/Support/TimeProfiler.cpp:137
           << escapeString(E.first)
-          << "\", \"args\":{ \"count\":" << CountPerName[E.first]
-          << ", \"avg ms\":" << (DurUs / CountPerName[E.first] / 1000)
-          << "} },\n";
+          << "\", \"args\":{ \"count\":" << CountAndTotalPerName[E.first].first
+          << ", \"avg ms\":"
----------------
takuto.ikuta wrote:
> I prefer to reuse result of lookup for map here in L139.
Ok.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60404