From 62439d182e6f9c9e0f0624c144fe81f5091a6322 Mon Sep 17 00:00:00 2001 From: Aiden Grossman Date: Tue, 21 Nov 2023 00:53:01 -0800 Subject: [PATCH] [NewPM] Remove MakeGuardsExplicitLegacyPass (#72946) This pass isn't used anywhere or tested anywhere upstream (it doesn't even have a create function), so remove it. --- llvm/include/llvm/InitializePasses.h | 1 - .../Transforms/Scalar/MakeGuardsExplicit.cpp | 20 ------------------- llvm/lib/Transforms/Scalar/Scalar.cpp | 1 - 3 files changed, 22 deletions(-) diff --git a/llvm/include/llvm/InitializePasses.h b/llvm/include/llvm/InitializePasses.h index bd81fbe35ca91..896ec849ae14d 100644 --- a/llvm/include/llvm/InitializePasses.h +++ b/llvm/include/llvm/InitializePasses.h @@ -113,7 +113,6 @@ void initializeExpandMemCmpPassPass(PassRegistry&); void initializeExpandPostRAPass(PassRegistry&); void initializeExpandReductionsPass(PassRegistry&); void initializeExpandVectorPredicationPass(PassRegistry &); -void initializeMakeGuardsExplicitLegacyPassPass(PassRegistry&); void initializeExternalAAWrapperPassPass(PassRegistry&); void initializeFEntryInserterPass(PassRegistry&); void initializeFinalizeISelPass(PassRegistry&); diff --git a/llvm/lib/Transforms/Scalar/MakeGuardsExplicit.cpp b/llvm/lib/Transforms/Scalar/MakeGuardsExplicit.cpp index a3f09a5a33c37..78e474f925b54 100644 --- a/llvm/lib/Transforms/Scalar/MakeGuardsExplicit.cpp +++ b/llvm/lib/Transforms/Scalar/MakeGuardsExplicit.cpp @@ -42,17 +42,6 @@ using namespace llvm; -namespace { -struct MakeGuardsExplicitLegacyPass : public FunctionPass { - static char ID; - MakeGuardsExplicitLegacyPass() : FunctionPass(ID) { - initializeMakeGuardsExplicitLegacyPassPass(*PassRegistry::getPassRegistry()); - } - - bool runOnFunction(Function &F) override; -}; -} - static void turnToExplicitForm(CallInst *Guard, Function *DeoptIntrinsic) { // Replace the guard with an explicit branch (just like in GuardWidening). BasicBlock *OriginalBB = Guard->getParent(); @@ -89,15 +78,6 @@ static bool explicifyGuards(Function &F) { return true; } -bool MakeGuardsExplicitLegacyPass::runOnFunction(Function &F) { - return explicifyGuards(F); -} - -char MakeGuardsExplicitLegacyPass::ID = 0; -INITIALIZE_PASS(MakeGuardsExplicitLegacyPass, "make-guards-explicit", - "Lower the guard intrinsic to explicit control flow form", - false, false) - PreservedAnalyses MakeGuardsExplicitPass::run(Function &F, FunctionAnalysisManager &) { if (explicifyGuards(F)) diff --git a/llvm/lib/Transforms/Scalar/Scalar.cpp b/llvm/lib/Transforms/Scalar/Scalar.cpp index 8fee751476263..d19f1c697d878 100644 --- a/llvm/lib/Transforms/Scalar/Scalar.cpp +++ b/llvm/lib/Transforms/Scalar/Scalar.cpp @@ -24,7 +24,6 @@ void llvm::initializeScalarOpts(PassRegistry &Registry) { initializeGVNLegacyPassPass(Registry); initializeEarlyCSELegacyPassPass(Registry); initializeEarlyCSEMemSSALegacyPassPass(Registry); - initializeMakeGuardsExplicitLegacyPassPass(Registry); initializeFlattenCFGLegacyPassPass(Registry); initializeInferAddressSpacesPass(Registry); initializeInstSimplifyLegacyPassPass(Registry);