[all-commits] [llvm/llvm-project] eef95f: [BrachProbablityInfo] Set edge probabilities at on...

Yevgeny Rouban via All-commits all-commits at lists.llvm.org
Tue May 12 23:56:28 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: eef95f2746c3347b8dad19091ffb82a88d73acd3
      https://github.com/llvm/llvm-project/commit/eef95f2746c3347b8dad19091ffb82a88d73acd3
  Author: Yevgeny Rouban <yrouban at azul.com>
  Date:   2020-05-13 (Wed, 13 May 2020)

  Changed paths:
    M llvm/include/llvm/Analysis/BranchProbabilityInfo.h
    M llvm/lib/Analysis/BranchProbabilityInfo.cpp
    M llvm/lib/Transforms/Scalar/JumpThreading.cpp
    M llvm/lib/Transforms/Utils/BreakCriticalEdges.cpp
    M llvm/lib/Transforms/Utils/CodeExtractor.cpp

  Log Message:
  -----------
  [BrachProbablityInfo] Set edge probabilities at once. NFC.

Hide the method that allows setting probability for particular
edge and introduce a public method that sets probabilities for
all outgoing edges at once.
Setting individual edge probability is error prone. More over
it is difficult to check that the total probability is 1.0
because there is no easy way to know when the user finished
setting all the probabilities.

Reviewers: yamauchi, ebrevnov
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D79396




More information about the All-commits mailing list