AMDGPU: Hoist subtarget lookup

arsenm · arsenm · commit 6c770a09bed7 · 2020-08-27T10:27:56.000-04:00
diff --git a/llvm/lib/Target/AMDGPU/AMDGPULowerIntrinsics.cpp b/llvm/lib/Target/AMDGPU/AMDGPULowerIntrinsics.cpp
@@ -124,14 +124,15 @@ bool AMDGPULowerIntrinsics::makeLIDRangeMetadata(Function &F) const {
     return false;
 
   const TargetMachine &TM = TPC->getTM<TargetMachine>();
+  const AMDGPUSubtarget &ST = AMDGPUSubtarget::get(TM, F);
   bool Changed = false;
 
   for (auto *U : F.users()) {
     auto *CI = dyn_cast<CallInst>(U);
     if (!CI)
       continue;
 
-    Changed |= AMDGPUSubtarget::get(TM, F).makeLIDRangeMetadata(CI);
+    Changed |= ST.makeLIDRangeMetadata(CI);
   }
   return Changed;
 }

Original file line number	Diff line number	Diff line change
`@@ -124,14 +124,15 @@ bool AMDGPULowerIntrinsics::makeLIDRangeMetadata(Function &F) const {`
`124`	`124`	`return false;`
`125`	`125`
`126`	`126`	`const TargetMachine &TM = TPC->getTM<TargetMachine>();`
	`127`	`+ const AMDGPUSubtarget &ST = AMDGPUSubtarget::get(TM, F);`
`127`	`128`	`bool Changed = false;`
`128`	`129`
`129`	`130`	`for (auto *U : F.users()) {`
`130`	`131`	`auto *CI = dyn_cast<CallInst>(U);`
`131`	`132`	`if (!CI)`
`132`	`133`	`continue;`
`133`	`134`
`134`		`- Changed \|= AMDGPUSubtarget::get(TM, F).makeLIDRangeMetadata(CI);`
	`135`	`+ Changed \|= ST.makeLIDRangeMetadata(CI);`
`135`	`136`	`}`
`136`	`137`	`return Changed;`
`137`	`138`	`}`