Skip to content

Commit 47928ab

Browse files
authored
[CodeGen][NewPM] Port RegUsageInfoPropagation pass to NPM (#114010)
1 parent d69cc05 commit 47928ab

File tree

8 files changed

+86
-26
lines changed

8 files changed

+86
-26
lines changed
Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
//===- llvm/CodeGen/RegUsageInfoPropagate.h ---------------------*- C++ -*-===//
2+
//
3+
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4+
// See https://llvm.org/LICENSE.txt for license information.
5+
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6+
//
7+
//===----------------------------------------------------------------------===//
8+
9+
#ifndef LLVM_CODEGEN_REGUSAGEINFOPROPAGATE_H
10+
#define LLVM_CODEGEN_REGUSAGEINFOPROPAGATE_H
11+
12+
#include "llvm/CodeGen/MachinePassManager.h"
13+
14+
namespace llvm {
15+
16+
class RegUsageInfoPropagationPass
17+
: public PassInfoMixin<RegUsageInfoPropagationPass> {
18+
public:
19+
PreservedAnalyses run(MachineFunction &MF,
20+
MachineFunctionAnalysisManager &MFAM);
21+
};
22+
23+
} // namespace llvm
24+
25+
#endif // LLVM_CODEGEN_REGUSAGEINFOPROPAGATE_H

llvm/include/llvm/InitializePasses.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ void initializeRegAllocScoringPass(PassRegistry &);
259259
void initializeRegBankSelectPass(PassRegistry &);
260260
void initializeRegToMemWrapperPassPass(PassRegistry &);
261261
void initializeRegUsageInfoCollectorLegacyPass(PassRegistry &);
262-
void initializeRegUsageInfoPropagationPass(PassRegistry &);
262+
void initializeRegUsageInfoPropagationLegacyPass(PassRegistry &);
263263
void initializeRegionInfoPassPass(PassRegistry &);
264264
void initializeRegionOnlyPrinterPass(PassRegistry &);
265265
void initializeRegionOnlyViewerPass(PassRegistry &);

llvm/include/llvm/Passes/CodeGenPassBuilder.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@
5555
#include "llvm/CodeGen/PreISelIntrinsicLowering.h"
5656
#include "llvm/CodeGen/RegAllocFast.h"
5757
#include "llvm/CodeGen/RegUsageInfoCollector.h"
58+
#include "llvm/CodeGen/RegUsageInfoPropagate.h"
5859
#include "llvm/CodeGen/RegisterUsageInfo.h"
5960
#include "llvm/CodeGen/ReplaceWithVeclib.h"
6061
#include "llvm/CodeGen/SafeStack.h"

llvm/include/llvm/Passes/MachinePassRegistry.def

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,6 +158,7 @@ MACHINE_FUNCTION_PASS("print<machine-post-dom-tree>",
158158
MACHINE_FUNCTION_PASS("print<slot-indexes>", SlotIndexesPrinterPass(dbgs()))
159159
MACHINE_FUNCTION_PASS("print<virtregmap>", VirtRegMapPrinterPass(dbgs()))
160160
MACHINE_FUNCTION_PASS("reg-usage-collector", RegUsageInfoCollectorPass())
161+
MACHINE_FUNCTION_PASS("reg-usage-propagation", RegUsageInfoPropagationPass())
161162
MACHINE_FUNCTION_PASS("require-all-machine-function-properties",
162163
RequireAllMachineFunctionPropertiesPass())
163164
MACHINE_FUNCTION_PASS("stack-coloring", StackColoringPass())
@@ -252,7 +253,6 @@ DUMMY_MACHINE_FUNCTION_PASS("prologepilog-code", PrologEpilogCodeInserterPass)
252253
DUMMY_MACHINE_FUNCTION_PASS("ra-basic", RABasicPass)
253254
DUMMY_MACHINE_FUNCTION_PASS("ra-greedy", RAGreedyPass)
254255
DUMMY_MACHINE_FUNCTION_PASS("ra-pbqp", RAPBQPPass)
255-
DUMMY_MACHINE_FUNCTION_PASS("reg-usage-propagation", RegUsageInfoPropagationPass)
256256
DUMMY_MACHINE_FUNCTION_PASS("regalloc", RegAllocPass)
257257
DUMMY_MACHINE_FUNCTION_PASS("regallocscoringpass", RegAllocScoringPass)
258258
DUMMY_MACHINE_FUNCTION_PASS("regbankselect", RegBankSelectPass)

llvm/lib/CodeGen/CodeGen.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ void llvm::initializeCodeGen(PassRegistry &Registry) {
114114
initializeRAGreedyPass(Registry);
115115
initializeRegAllocFastPass(Registry);
116116
initializeRegUsageInfoCollectorLegacyPass(Registry);
117-
initializeRegUsageInfoPropagationPass(Registry);
117+
initializeRegUsageInfoPropagationLegacyPass(Registry);
118118
initializeRegisterCoalescerPass(Registry);
119119
initializeRemoveLoadsIntoFakeUsesPass(Registry);
120120
initializeRemoveRedundantDebugValuesPass(Registry);

llvm/lib/CodeGen/RegUsageInfoPropagate.cpp

Lines changed: 52 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,18 @@
1818
///
1919
//===----------------------------------------------------------------------===//
2020

21+
#include "llvm/CodeGen/RegUsageInfoPropagate.h"
2122
#include "llvm/CodeGen/MachineBasicBlock.h"
2223
#include "llvm/CodeGen/MachineFrameInfo.h"
2324
#include "llvm/CodeGen/MachineFunctionPass.h"
2425
#include "llvm/CodeGen/MachineInstr.h"
2526
#include "llvm/CodeGen/MachineRegisterInfo.h"
2627
#include "llvm/CodeGen/Passes.h"
2728
#include "llvm/CodeGen/RegisterUsageInfo.h"
29+
#include "llvm/IR/Analysis.h"
2830
#include "llvm/IR/Module.h"
2931
#include "llvm/Pass.h"
32+
#include "llvm/Passes/CodeGenPassBuilder.h"
3033
#include "llvm/Support/Debug.h"
3134
#include "llvm/Support/raw_ostream.h"
3235

@@ -38,26 +41,16 @@ using namespace llvm;
3841

3942
namespace {
4043

41-
class RegUsageInfoPropagation : public MachineFunctionPass {
44+
class RegUsageInfoPropagation {
4245
public:
43-
RegUsageInfoPropagation() : MachineFunctionPass(ID) {
44-
PassRegistry &Registry = *PassRegistry::getPassRegistry();
45-
initializeRegUsageInfoPropagationPass(Registry);
46-
}
46+
explicit RegUsageInfoPropagation(PhysicalRegisterUsageInfo *PRUI)
47+
: PRUI(PRUI) {}
4748

48-
StringRef getPassName() const override { return RUIP_NAME; }
49-
50-
bool runOnMachineFunction(MachineFunction &MF) override;
51-
52-
void getAnalysisUsage(AnalysisUsage &AU) const override {
53-
AU.addRequired<PhysicalRegisterUsageInfoWrapperLegacy>();
54-
AU.setPreservesAll();
55-
MachineFunctionPass::getAnalysisUsage(AU);
56-
}
57-
58-
static char ID;
49+
bool run(MachineFunction &MF);
5950

6051
private:
52+
PhysicalRegisterUsageInfo *PRUI;
53+
6154
static void setRegMask(MachineInstr &MI, ArrayRef<uint32_t> RegMask) {
6255
assert(RegMask.size() ==
6356
MachineOperand::getRegMaskSize(MI.getParent()->getParent()
@@ -71,15 +64,34 @@ class RegUsageInfoPropagation : public MachineFunctionPass {
7164
}
7265
};
7366

67+
class RegUsageInfoPropagationLegacy : public MachineFunctionPass {
68+
public:
69+
static char ID;
70+
RegUsageInfoPropagationLegacy() : MachineFunctionPass(ID) {
71+
PassRegistry &Registry = *PassRegistry::getPassRegistry();
72+
initializeRegUsageInfoPropagationLegacyPass(Registry);
73+
}
74+
75+
StringRef getPassName() const override { return RUIP_NAME; }
76+
77+
bool runOnMachineFunction(MachineFunction &MF) override;
78+
79+
void getAnalysisUsage(AnalysisUsage &AU) const override {
80+
AU.addRequired<PhysicalRegisterUsageInfoWrapperLegacy>();
81+
AU.setPreservesAll();
82+
MachineFunctionPass::getAnalysisUsage(AU);
83+
}
84+
};
85+
7486
} // end of anonymous namespace
7587

76-
INITIALIZE_PASS_BEGIN(RegUsageInfoPropagation, "reg-usage-propagation",
88+
INITIALIZE_PASS_BEGIN(RegUsageInfoPropagationLegacy, "reg-usage-propagation",
7789
RUIP_NAME, false, false)
7890
INITIALIZE_PASS_DEPENDENCY(PhysicalRegisterUsageInfoWrapperLegacy)
79-
INITIALIZE_PASS_END(RegUsageInfoPropagation, "reg-usage-propagation",
91+
INITIALIZE_PASS_END(RegUsageInfoPropagationLegacy, "reg-usage-propagation",
8092
RUIP_NAME, false, false)
8193

82-
char RegUsageInfoPropagation::ID = 0;
94+
char RegUsageInfoPropagationLegacy::ID = 0;
8395

8496
// Assumes call instructions have a single reference to a function.
8597
static const Function *findCalledFunction(const Module &M,
@@ -95,12 +107,29 @@ static const Function *findCalledFunction(const Module &M,
95107
return nullptr;
96108
}
97109

98-
bool RegUsageInfoPropagation::runOnMachineFunction(MachineFunction &MF) {
99-
const Module &M = *MF.getFunction().getParent();
110+
bool RegUsageInfoPropagationLegacy::runOnMachineFunction(MachineFunction &MF) {
100111
PhysicalRegisterUsageInfo *PRUI =
101112
&getAnalysis<PhysicalRegisterUsageInfoWrapperLegacy>().getPRUI();
102113

103-
LLVM_DEBUG(dbgs() << " ++++++++++++++++++++ " << getPassName()
114+
RegUsageInfoPropagation RUIP(PRUI);
115+
return RUIP.run(MF);
116+
}
117+
118+
PreservedAnalyses
119+
RegUsageInfoPropagationPass::run(MachineFunction &MF,
120+
MachineFunctionAnalysisManager &MFAM) {
121+
Module &MFA = *MF.getFunction().getParent();
122+
auto *PRUI = MFAM.getResult<ModuleAnalysisManagerMachineFunctionProxy>(MF)
123+
.getCachedResult<PhysicalRegisterUsageAnalysis>(MFA);
124+
assert(PRUI && "PhysicalRegisterUsageAnalysis not available");
125+
RegUsageInfoPropagation(PRUI).run(MF);
126+
return PreservedAnalyses::all();
127+
}
128+
129+
bool RegUsageInfoPropagation::run(MachineFunction &MF) {
130+
const Module &M = *MF.getFunction().getParent();
131+
132+
LLVM_DEBUG(dbgs() << " ++++++++++++++++++++ " << RUIP_NAME
104133
<< " ++++++++++++++++++++ \n");
105134
LLVM_DEBUG(dbgs() << "MachineFunction : " << MF.getName() << "\n");
106135

@@ -151,5 +180,5 @@ bool RegUsageInfoPropagation::runOnMachineFunction(MachineFunction &MF) {
151180
}
152181

153182
FunctionPass *llvm::createRegUsageInfoPropPass() {
154-
return new RegUsageInfoPropagation();
183+
return new RegUsageInfoPropagationLegacy();
155184
}

llvm/lib/Passes/PassBuilder.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,7 @@
121121
#include "llvm/CodeGen/PreISelIntrinsicLowering.h"
122122
#include "llvm/CodeGen/RegAllocFast.h"
123123
#include "llvm/CodeGen/RegUsageInfoCollector.h"
124+
#include "llvm/CodeGen/RegUsageInfoPropagate.h"
124125
#include "llvm/CodeGen/RegisterUsageInfo.h"
125126
#include "llvm/CodeGen/SafeStack.h"
126127
#include "llvm/CodeGen/SelectOptimize.h"

llvm/test/CodeGen/AArch64/preserve.ll

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11

22
; RUN: llc -enable-ipra -print-regusage -o /dev/null 2>&1 < %s | FileCheck %s
33

4+
; RUN: llc -stop-after=prologepilog -o - %s \
5+
; RUN: | llc -x=mir -enable-ipra -passes="require<reg-usage>,function(machine-function(reg-usage-propagation,reg-usage-collector)),print<reg-usage>" -o /dev/null 2>&1 \
6+
; RUN: | FileCheck %s
7+
48
target triple = "aarch64-unknown-unknown"
59
declare void @bar1()
610
define preserve_mostcc void @baz() #0 {

0 commit comments

Comments
 (0)