Skip to content

Commit 27fade6

Browse files
committed
AMDGPU: Use GCNTargetMachine in AMDGPUCodeGenPassBuilder
R600 has a separate CodeGenPassBuilder anyway.
1 parent 5286656 commit 27fade6

File tree

4 files changed

+18
-18
lines changed

4 files changed

+18
-18
lines changed

llvm/lib/Target/AMDGPU/AMDGPUCodeGenPassBuilder.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
using namespace llvm;
1717

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

llvm/lib/Target/AMDGPU/AMDGPUCodeGenPassBuilder.h

+3-3
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

+8-8
Original file line numberDiff line numberDiff line change
@@ -656,14 +656,6 @@ parseAMDGPUAtomicOptimizerStrategy(StringRef Params) {
656656
return make_error<StringError>("invalid parameter", inconvertibleErrorCode());
657657
}
658658

659-
Error AMDGPUTargetMachine::buildCodeGenPipeline(
660-
ModulePassManager &MPM, raw_pwrite_stream &Out, raw_pwrite_stream *DwoOut,
661-
CodeGenFileType FileType, const CGPassBuilderOption &Opts,
662-
PassInstrumentationCallbacks *PIC) {
663-
AMDGPUCodeGenPassBuilder CGPB(*this, Opts, PIC);
664-
return CGPB.buildPipeline(MPM, Out, DwoOut, FileType);
665-
}
666-
667659
Expected<AMDGPUAttributorOptions>
668660
parseAMDGPUAttributorPassOptions(StringRef Params) {
669661
AMDGPUAttributorOptions Result;
@@ -920,6 +912,14 @@ GCNTargetMachine::getTargetTransformInfo(const Function &F) const {
920912
return TargetTransformInfo(GCNTTIImpl(this, F));
921913
}
922914

915+
Error GCNTargetMachine::buildCodeGenPipeline(
916+
ModulePassManager &MPM, raw_pwrite_stream &Out, raw_pwrite_stream *DwoOut,
917+
CodeGenFileType FileType, const CGPassBuilderOption &Opts,
918+
PassInstrumentationCallbacks *PIC) {
919+
AMDGPUCodeGenPassBuilder CGPB(*this, Opts, PIC);
920+
return CGPB.buildPipeline(MPM, Out, DwoOut, FileType);
921+
}
922+
923923
//===----------------------------------------------------------------------===//
924924
// AMDGPU Pass Setup
925925
//===----------------------------------------------------------------------===//

llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.h

+6-6
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)