[PATCH] D56464: Refactor synthetic profile count computation. NFC.

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 8 16:47:59 PST 2019


davidxl added inline comments.


================
Comment at: lib/Analysis/SyntheticCountsUtils.cpp:64
     auto Callee = CGT::edge_dest(E.second);
-    RelFreq *= Scaled64(GetCount(Caller), 0);
-    uint64_t AdditionalCount = RelFreq.toInt<uint64_t>();
     AdditionalCounts[Callee] += AdditionalCount;
   }
----------------
To avoid overflow, AdditionalCounts be changed to a map to Scaled64?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56464





More information about the llvm-commits mailing list