Skip to content

Commit 0446b40

Browse files
authored
[NFC][AMDGPU][Attributor] Only iterate over filtered functions when creating AAs (#108417)
1 parent 7a6945f commit 0446b40

File tree

1 file changed

+8
-11
lines changed

1 file changed

+8
-11
lines changed

llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp

+8-11
Original file line numberDiff line numberDiff line change
@@ -1065,19 +1065,16 @@ static bool runImpl(Module &M, AnalysisGetter &AG, TargetMachine &TM,
10651065

10661066
Attributor A(Functions, InfoCache, AC);
10671067

1068-
for (Function &F : M) {
1069-
if (F.isIntrinsic())
1070-
continue;
1071-
1072-
A.getOrCreateAAFor<AAAMDAttributes>(IRPosition::function(F));
1073-
A.getOrCreateAAFor<AAUniformWorkGroupSize>(IRPosition::function(F));
1074-
A.getOrCreateAAFor<AAAMDGPUNoAGPR>(IRPosition::function(F));
1075-
CallingConv::ID CC = F.getCallingConv();
1068+
for (auto *F : Functions) {
1069+
A.getOrCreateAAFor<AAAMDAttributes>(IRPosition::function(*F));
1070+
A.getOrCreateAAFor<AAUniformWorkGroupSize>(IRPosition::function(*F));
1071+
A.getOrCreateAAFor<AAAMDGPUNoAGPR>(IRPosition::function(*F));
1072+
CallingConv::ID CC = F->getCallingConv();
10761073
if (!AMDGPU::isEntryFunctionCC(CC)) {
1077-
A.getOrCreateAAFor<AAAMDFlatWorkGroupSize>(IRPosition::function(F));
1078-
A.getOrCreateAAFor<AAAMDWavesPerEU>(IRPosition::function(F));
1074+
A.getOrCreateAAFor<AAAMDFlatWorkGroupSize>(IRPosition::function(*F));
1075+
A.getOrCreateAAFor<AAAMDWavesPerEU>(IRPosition::function(*F));
10791076
} else if (CC == CallingConv::AMDGPU_KERNEL) {
1080-
addPreloadKernArgHint(F, TM);
1077+
addPreloadKernArgHint(*F, TM);
10811078
}
10821079

10831080
for (auto &I : instructions(F)) {

0 commit comments

Comments
 (0)