Skip to content

Commit

Permalink
Revert r338431: "Add DebugCounters to DivRemPairs"
Browse files Browse the repository at this point in the history
This reverts r338431; the test it added is making buildbots unhappy.
Locally, I can repro the failure on reverse-iteration builds.

llvm-svn: 338442
  • Loading branch information
gburgessiv committed Jul 31, 2018
1 parent 67d393f commit 497e8fa
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 96 deletions.
6 changes: 0 additions & 6 deletions llvm/lib/Transforms/Scalar/DivRemPairs.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
#include "llvm/IR/Dominators.h"
#include "llvm/IR/Function.h"
#include "llvm/Pass.h"
#include "llvm/Support/DebugCounter.h"
#include "llvm/Transforms/Scalar.h"
#include "llvm/Transforms/Utils/BypassSlowDivision.h"
using namespace llvm;
Expand All @@ -30,8 +29,6 @@ using namespace llvm;
STATISTIC(NumPairs, "Number of div/rem pairs");
STATISTIC(NumHoisted, "Number of instructions hoisted");
STATISTIC(NumDecomposed, "Number of instructions decomposed");
DEBUG_COUNTER(DRPCounter, "div-rem-pairs-transform",
"Controls transformations in div-rem-pairs pass");

/// Find matching pairs of integer div/rem ops (they have the same numerator,
/// denominator, and signedness). If they exist in different basic blocks, bring
Expand Down Expand Up @@ -96,9 +93,6 @@ static bool optimizeDivRem(Function &F, const TargetTransformInfo &TTI,
if (!DivDominates && !DT.dominates(RemInst, DivInst))
continue;

if (!DebugCounter::shouldExecute(DRPCounter))
continue;

if (HasDivRemOp) {
// The target has a single div/rem operation. Hoist the lower instruction
// to make the matched pair visible to the backend.
Expand Down
90 changes: 0 additions & 90 deletions llvm/test/Other/debugcounter-divrempairs.ll

This file was deleted.

0 comments on commit 497e8fa

Please sign in to comment.