Skip to content

Commit

Permalink
[NewPM] Remove LowerWidenableConditionLegacyPass (#72818)
Browse files Browse the repository at this point in the history
This legacy pass isn't used anywhere upstream and thus has no test
coverage, so remove it.
  • Loading branch information
boomanaiden154 committed Nov 20, 2023
1 parent 72ab995 commit 4671f18
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 35 deletions.
1 change: 0 additions & 1 deletion llvm/include/llvm/InitializePasses.h
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,6 @@ void initializeLowerAtomicLegacyPassPass(PassRegistry&);
void initializeLowerConstantIntrinsicsPass(PassRegistry&);
void initializeLowerEmuTLSPass(PassRegistry&);
void initializeLowerGlobalDtorsLegacyPassPass(PassRegistry &);
void initializeLowerWidenableConditionLegacyPassPass(PassRegistry&);
void initializeLowerIntrinsicsPass(PassRegistry&);
void initializeLowerInvokeLegacyPassPass(PassRegistry&);
void initializeLowerSwitchLegacyPassPass(PassRegistry &);
Expand Down
6 changes: 0 additions & 6 deletions llvm/include/llvm/Transforms/Scalar.h
Original file line number Diff line number Diff line change
Expand Up @@ -165,12 +165,6 @@ FunctionPass *createSinkingPass();
//
Pass *createLowerAtomicPass();

//===----------------------------------------------------------------------===//
//
// LowerWidenableCondition - Lower widenable condition to i1 true.
//
Pass *createLowerWidenableConditionPass();

//===----------------------------------------------------------------------===//
//
// MergeICmps - Merge integer comparison chains into a memcmp
Expand Down
27 changes: 0 additions & 27 deletions llvm/lib/Transforms/Scalar/LowerWidenableCondition.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -19,24 +19,10 @@
#include "llvm/IR/Intrinsics.h"
#include "llvm/IR/Module.h"
#include "llvm/IR/PatternMatch.h"
#include "llvm/InitializePasses.h"
#include "llvm/Pass.h"
#include "llvm/Transforms/Scalar.h"

using namespace llvm;

namespace {
struct LowerWidenableConditionLegacyPass : public FunctionPass {
static char ID;
LowerWidenableConditionLegacyPass() : FunctionPass(ID) {
initializeLowerWidenableConditionLegacyPassPass(
*PassRegistry::getPassRegistry());
}

bool runOnFunction(Function &F) override;
};
}

static bool lowerWidenableCondition(Function &F) {
// Check if we can cheaply rule out the possibility of not having any work to
// do.
Expand Down Expand Up @@ -65,19 +51,6 @@ static bool lowerWidenableCondition(Function &F) {
return true;
}

bool LowerWidenableConditionLegacyPass::runOnFunction(Function &F) {
return lowerWidenableCondition(F);
}

char LowerWidenableConditionLegacyPass::ID = 0;
INITIALIZE_PASS(LowerWidenableConditionLegacyPass, "lower-widenable-condition",
"Lower the widenable condition to default true value", false,
false)

Pass *llvm::createLowerWidenableConditionPass() {
return new LowerWidenableConditionLegacyPass();
}

PreservedAnalyses LowerWidenableConditionPass::run(Function &F,
FunctionAnalysisManager &AM) {
if (lowerWidenableCondition(F))
Expand Down
1 change: 0 additions & 1 deletion llvm/lib/Transforms/Scalar/Scalar.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ void llvm::initializeScalarOpts(PassRegistry &Registry) {
initializeLoopUnrollPass(Registry);
initializeLowerAtomicLegacyPassPass(Registry);
initializeLowerConstantIntrinsicsPass(Registry);
initializeLowerWidenableConditionLegacyPassPass(Registry);
initializeMergeICmpsLegacyPassPass(Registry);
initializeNaryReassociateLegacyPassPass(Registry);
initializePartiallyInlineLibCallsLegacyPassPass(Registry);
Expand Down

0 comments on commit 4671f18

Please sign in to comment.