Skip to content

Commit 835e2e4

Browse files
[NewPM] Remove CFGViewerLegacyPass (#73410)
This pass isn't used anywhere upstream, has a NewPM equivalent, and has no test coverage. For these reasons, remove it.
1 parent 631e737 commit 835e2e4

File tree

3 files changed

+0
-33
lines changed

3 files changed

+0
-33
lines changed

llvm/include/llvm/InitializePasses.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,6 @@ void initializeCFGPrinterLegacyPassPass(PassRegistry&);
7171
void initializeCFGSimplifyPassPass(PassRegistry&);
7272
void initializeCFGuardPass(PassRegistry&);
7373
void initializeCFGuardLongjmpPass(PassRegistry&);
74-
void initializeCFGViewerLegacyPassPass(PassRegistry&);
7574
void initializeCFIFixupPass(PassRegistry&);
7675
void initializeCFIInstrInserterPass(PassRegistry&);
7776
void initializeCallBrPreparePass(PassRegistry &);

llvm/lib/Analysis/Analysis.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ void llvm::initializeAnalysis(PassRegistry &Registry) {
2424
initializeCallGraphWrapperPassPass(Registry);
2525
initializeCallGraphDOTPrinterPass(Registry);
2626
initializeCallGraphViewerPass(Registry);
27-
initializeCFGViewerLegacyPassPass(Registry);
2827
initializeCFGPrinterLegacyPassPass(Registry);
2928
initializeCFGOnlyViewerLegacyPassPass(Registry);
3029
initializeCFGOnlyPrinterLegacyPassPass(Registry);

llvm/lib/Analysis/CFGPrinter.cpp

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -92,37 +92,6 @@ static void viewCFG(Function &F, const BlockFrequencyInfo *BFI,
9292
ViewGraph(&CFGInfo, "cfg." + F.getName(), CFGOnly);
9393
}
9494

95-
namespace {
96-
struct CFGViewerLegacyPass : public FunctionPass {
97-
static char ID; // Pass identifcation, replacement for typeid
98-
CFGViewerLegacyPass() : FunctionPass(ID) {
99-
initializeCFGViewerLegacyPassPass(*PassRegistry::getPassRegistry());
100-
}
101-
102-
bool runOnFunction(Function &F) override {
103-
if (!CFGFuncName.empty() && !F.getName().contains(CFGFuncName))
104-
return false;
105-
auto *BPI = &getAnalysis<BranchProbabilityInfoWrapperPass>().getBPI();
106-
auto *BFI = &getAnalysis<BlockFrequencyInfoWrapperPass>().getBFI();
107-
viewCFG(F, BFI, BPI, getMaxFreq(F, BFI));
108-
return false;
109-
}
110-
111-
void print(raw_ostream &OS, const Module * = nullptr) const override {}
112-
113-
void getAnalysisUsage(AnalysisUsage &AU) const override {
114-
FunctionPass::getAnalysisUsage(AU);
115-
AU.addRequired<BlockFrequencyInfoWrapperPass>();
116-
AU.addRequired<BranchProbabilityInfoWrapperPass>();
117-
AU.setPreservesAll();
118-
}
119-
};
120-
} // namespace
121-
122-
char CFGViewerLegacyPass::ID = 0;
123-
INITIALIZE_PASS(CFGViewerLegacyPass, "view-cfg", "View CFG of function", false,
124-
true)
125-
12695
PreservedAnalyses CFGViewerPass::run(Function &F, FunctionAnalysisManager &AM) {
12796
if (!CFGFuncName.empty() && !F.getName().contains(CFGFuncName))
12897
return PreservedAnalyses::all();

0 commit comments

Comments
 (0)