[PATCH] D12603: Use fixed-point representation for BranchProbability

Cong Hou via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 10 12:21:01 PDT 2015


congh updated this revision to Diff 34474.
congh added a comment.

Update the patch by fixing block frequency / branch probability unit tests failures.

I am not sure if this is the right way to update the unit tests. The fixed-point representation has precision issues comparing to the previous rational representation and that is why we need to update the comparison right answers in unit tests.


http://reviews.llvm.org/D12603

Files:
  include/llvm/Analysis/BlockFrequencyInfoImpl.h
  include/llvm/Support/BranchProbability.h
  lib/Support/BranchProbability.cpp
  lib/Target/ARM/ARMBaseInstrInfo.cpp
  test/Analysis/BlockFrequencyInfo/basic.ll
  test/Analysis/BlockFrequencyInfo/loops_with_profile_info.ll
  test/Analysis/BranchProbabilityInfo/basic.ll
  test/Analysis/BranchProbabilityInfo/loop.ll
  test/Analysis/BranchProbabilityInfo/noreturn.ll
  test/Analysis/BranchProbabilityInfo/pr18705.ll
  test/Analysis/BranchProbabilityInfo/pr22718.ll
  test/CodeGen/AArch64/fast-isel-branch-cond-split.ll
  test/CodeGen/ARM/2013-10-11-select-stalls.ll
  test/CodeGen/ARM/ifcvt4.ll
  test/CodeGen/ARM/sjlj-prepare-critical-edge.ll
  test/CodeGen/ARM/tail-merge-branch-weight.ll
  test/CodeGen/ARM/test-sharedidx.ll
  test/CodeGen/Thumb2/thumb2-ifcvt1.ll
  test/Transforms/SampleProfile/branch.ll
  test/Transforms/SampleProfile/calls.ll
  test/Transforms/SampleProfile/discriminator.ll
  test/Transforms/SampleProfile/fnptr.ll
  test/Transforms/SampleProfile/propagate.ll
  unittests/Support/BlockFrequencyTest.cpp
  unittests/Support/BranchProbabilityTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12603.34474.patch
Type: text/x-patch
Size: 69655 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150910/c3ba297f/attachment.bin>


More information about the llvm-commits mailing list