Skip to content

Commit 0a6b6ee

Browse files
committed
AMDGPU: Use GCNTargetMachine in AMDGPUCodeGenPassBuilder
R600 has a separate CodeGenPassBuilder anyway.
1 parent 0fa3fea commit 0a6b6ee

File tree

4 files changed

+18
-18
lines changed

4 files changed

+18
-18
lines changed

llvm/lib/Target/AMDGPU/AMDGPUCodeGenPassBuilder.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
using namespace llvm;
1616

1717
AMDGPUCodeGenPassBuilder::AMDGPUCodeGenPassBuilder(
18-
AMDGPUTargetMachine &TM, const CGPassBuilderOption &Opts,
18+
GCNTargetMachine &TM, const CGPassBuilderOption &Opts,
1919
PassInstrumentationCallbacks *PIC)
2020
: CodeGenPassBuilder(TM, Opts, PIC) {
2121
Opt.RequiresCodeGenSCCOrder = true;

llvm/lib/Target/AMDGPU/AMDGPUCodeGenPassBuilder.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,12 @@
1414

1515
namespace llvm {
1616

17-
class AMDGPUTargetMachine;
17+
class GCNTargetMachine;
1818

1919
class AMDGPUCodeGenPassBuilder
20-
: public CodeGenPassBuilder<AMDGPUCodeGenPassBuilder, AMDGPUTargetMachine> {
20+
: public CodeGenPassBuilder<AMDGPUCodeGenPassBuilder, GCNTargetMachine> {
2121
public:
22-
AMDGPUCodeGenPassBuilder(AMDGPUTargetMachine &TM,
22+
AMDGPUCodeGenPassBuilder(GCNTargetMachine &TM,
2323
const CGPassBuilderOption &Opts,
2424
PassInstrumentationCallbacks *PIC);
2525

llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -660,14 +660,6 @@ parseAMDGPUAtomicOptimizerStrategy(StringRef Params) {
660660
return make_error<StringError>("invalid parameter", inconvertibleErrorCode());
661661
}
662662

663-
Error AMDGPUTargetMachine::buildCodeGenPipeline(
664-
ModulePassManager &MPM, raw_pwrite_stream &Out, raw_pwrite_stream *DwoOut,
665-
CodeGenFileType FileType, const CGPassBuilderOption &Opts,
666-
PassInstrumentationCallbacks *PIC) {
667-
AMDGPUCodeGenPassBuilder CGPB(*this, Opts, PIC);
668-
return CGPB.buildPipeline(MPM, Out, DwoOut, FileType);
669-
}
670-
671663
void AMDGPUTargetMachine::registerPassBuilderCallbacks(PassBuilder &PB) {
672664

673665
#define GET_PASS_REGISTRY "AMDGPUPassRegistry.def"
@@ -900,6 +892,14 @@ GCNTargetMachine::getTargetTransformInfo(const Function &F) const {
900892
return TargetTransformInfo(GCNTTIImpl(this, F));
901893
}
902894

895+
Error GCNTargetMachine::buildCodeGenPipeline(
896+
ModulePassManager &MPM, raw_pwrite_stream &Out, raw_pwrite_stream *DwoOut,
897+
CodeGenFileType FileType, const CGPassBuilderOption &Opts,
898+
PassInstrumentationCallbacks *PIC) {
899+
AMDGPUCodeGenPassBuilder CGPB(*this, Opts, PIC);
900+
return CGPB.buildPipeline(MPM, Out, DwoOut, FileType);
901+
}
902+
903903
//===----------------------------------------------------------------------===//
904904
// AMDGPU Pass Setup
905905
//===----------------------------------------------------------------------===//

llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,12 +52,6 @@ class AMDGPUTargetMachine : public LLVMTargetMachine {
5252
return TLOF.get();
5353
}
5454

55-
Error buildCodeGenPipeline(ModulePassManager &MPM, raw_pwrite_stream &Out,
56-
raw_pwrite_stream *DwoOut,
57-
CodeGenFileType FileType,
58-
const CGPassBuilderOption &Opts,
59-
PassInstrumentationCallbacks *PIC) override;
60-
6155
void registerPassBuilderCallbacks(PassBuilder &PB) override;
6256
void registerDefaultAliasAnalyses(AAManager &) override;
6357

@@ -103,6 +97,12 @@ class GCNTargetMachine final : public AMDGPUTargetMachine {
10397
return true;
10498
}
10599

100+
Error buildCodeGenPipeline(ModulePassManager &MPM, raw_pwrite_stream &Out,
101+
raw_pwrite_stream *DwoOut,
102+
CodeGenFileType FileType,
103+
const CGPassBuilderOption &Opts,
104+
PassInstrumentationCallbacks *PIC) override;
105+
106106
void registerMachineRegisterInfoCallback(MachineFunction &MF) const override;
107107

108108
MachineFunctionInfo *

0 commit comments

Comments
 (0)