Using branch probability to guide critical edge splitting.
authorDehao Chen <dehao@google.com>
Thu, 20 Oct 2016 18:06:52 +0000 (18:06 +0000)
committerDehao Chen <dehao@google.com>
Thu, 20 Oct 2016 18:06:52 +0000 (18:06 +0000)
commitf03f51555a6314c2af53394fcdf10ea538b0178d
treed484fd9c16a6867ad6590f7521a369c1b89d555e
parent53a7844383faecb8c3e0e7133bdfa8fdc0425f93
Using branch probability to guide critical edge splitting.

Summary:
The original heuristic to break critical edge during machine sink is relatively conservertive: when there is only one instruction sinkable to the critical edge, it is likely that the machine sink pass will not break the critical edge. This leads to many speculative instructions executed at runtime. However, with profile info, we could model the splitting benefits: if the critical edge has 50% taken rate, it would always be beneficial to split the critical edge to avoid the speculated runtime instructions. This patch uses profile to guide critical edge splitting in machine sink pass.

The performance impact on speccpu2006 on Intel sandybridge machines:

spec/2006/fp/C++/444.namd                  25.3  +0.26%
spec/2006/fp/C++/447.dealII               45.96  -0.10%
spec/2006/fp/C++/450.soplex               41.97  +1.49%
spec/2006/fp/C++/453.povray               36.83  -0.96%
spec/2006/fp/C/433.milc                   23.81  +0.32%
spec/2006/fp/C/470.lbm                    41.17  +0.34%
spec/2006/fp/C/482.sphinx3                48.13  +0.69%
spec/2006/int/C++/471.omnetpp             22.45  +3.25%
spec/2006/int/C++/473.astar               21.35  -2.06%
spec/2006/int/C++/483.xalancbmk           36.02  -2.39%
spec/2006/int/C/400.perlbench              33.7  -0.17%
spec/2006/int/C/401.bzip2                  22.9  +0.52%
spec/2006/int/C/403.gcc                   32.42  -0.54%
spec/2006/int/C/429.mcf                   39.59  +0.19%
spec/2006/int/C/445.gobmk                 26.98  -0.00%
spec/2006/int/C/456.hmmer                 24.52  -0.18%
spec/2006/int/C/458.sjeng                 28.26  +0.02%
spec/2006/int/C/462.libquantum            55.44  +3.74%
spec/2006/int/C/464.h264ref               46.67  -0.39%

geometric mean                                   +0.20%

Manually checked 473 and 471 to verify the diff is in the noise range.

Reviewers: rengolin, davidxl

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D24818

llvm-svn: 284757
llvm/lib/CodeGen/MachineSink.cpp
llvm/test/CodeGen/ARM/atomic-cmpxchg.ll
llvm/test/CodeGen/ARM/code-placement.ll
llvm/test/CodeGen/X86/block-placement.ll
llvm/test/CodeGen/X86/clz.ll
llvm/test/CodeGen/X86/loop-search.ll
llvm/test/CodeGen/X86/machine-sink.ll [new file with mode: 0644]
llvm/test/CodeGen/X86/phys_subreg_coalesce-2.ll
llvm/test/CodeGen/X86/pr2659.ll
llvm/test/DebugInfo/COFF/pieces.ll