[PATCH] D27862: [BPI] Use saturating multiplication to calculate denominators

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 16 15:32:16 PST 2016


vsk retitled this revision from "[BPI] Use saturating multiplication to calculate denominators (WIP)" to "[BPI] Use saturating multiplication to calculate denominators".
vsk updated the summary for this revision.
vsk updated this revision to Diff 81806.
vsk added a comment.
Herald added a subscriber: mgorny.

- Remove the lit test in favor of a more flexible lit test.
- Still no test for the cold call heuristic, because that would require creating ~2^26 edges. That's too much overhead for a unit test.


https://reviews.llvm.org/D27862

Files:
  lib/Analysis/BranchProbabilityInfo.cpp
  unittests/Analysis/BranchProbabilityInfoTest.cpp
  unittests/Analysis/CMakeLists.txt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27862.81806.patch
Type: text/x-patch
Size: 5701 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161216/4dbe30be/attachment.bin>


More information about the llvm-commits mailing list