Revert "[ControlHeightReduction] Don't combine a "poison" branch"

This reverts commit 38a64aa. llvm-clang-x86_64-expensive-checks-debian is failing: https://lab.llvm.org/buildbot/#/builders/16/builds/44249
llvm · Mar 1, 2023 · 368c9f8 · 368c9f8
1 parent c1c29eb
commit 368c9f8
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 77 deletions.
diff --git a/llvm/lib/Transforms/Instrumentation/ControlHeightReduction.cpp b/llvm/lib/Transforms/Instrumentation/ControlHeightReduction.cpp
@@ -631,7 +631,7 @@ static bool checkBiasedBranch(BranchInst *BI, Region *R,
                               DenseSet<Region *> &TrueBiasedRegionsGlobal,
                               DenseSet<Region *> &FalseBiasedRegionsGlobal,
                               DenseMap<Region *, BranchProbability> &BranchBiasMap) {
-  if (!BI->isConditional() || isa<PoisonValue>(BI->getCondition()))
+  if (!BI->isConditional())
     return false;
   BranchProbability ThenProb, ElseProb;
   if (!extractBranchProbabilities(BI, ThenProb, ElseProb))

diff --git a/llvm/test/Transforms/PGOProfile/chr-poison.ll b/llvm/test/Transforms/PGOProfile/chr-poison.ll