llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-backend-amdgpu Author: Shilei Tian (shiltian) <details> <summary>Changes</summary> --- Full diff: https://github.com/llvm/llvm-project/pull/101591.diff 1 Files Affected: - (modified) llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp (+12-11) ``````````diff diff --git a/llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp b/llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp index 9d3c9e1e2ef9f..de1f3421cce4e 100644 --- a/llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp +++ b/llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp @@ -1051,17 +1051,18 @@ static bool runImpl(Module &M, AnalysisGetter &AG, TargetMachine &TM) { Attributor A(Functions, InfoCache, AC); for (Function &F : M) { - if (!F.isIntrinsic()) { - A.getOrCreateAAFor<AAAMDAttributes>(IRPosition::function(F)); - A.getOrCreateAAFor<AAUniformWorkGroupSize>(IRPosition::function(F)); - A.getOrCreateAAFor<AAAMDGPUNoAGPR>(IRPosition::function(F)); - CallingConv::ID CC = F.getCallingConv(); - if (!AMDGPU::isEntryFunctionCC(CC)) { - A.getOrCreateAAFor<AAAMDFlatWorkGroupSize>(IRPosition::function(F)); - A.getOrCreateAAFor<AAAMDWavesPerEU>(IRPosition::function(F)); - } else if (CC == CallingConv::AMDGPU_KERNEL) { - addPreloadKernArgHint(F, TM); - } + if (F.isIntrinsic()) + continue; + + A.getOrCreateAAFor<AAAMDAttributes>(IRPosition::function(F)); + A.getOrCreateAAFor<AAUniformWorkGroupSize>(IRPosition::function(F)); + A.getOrCreateAAFor<AAAMDGPUNoAGPR>(IRPosition::function(F)); + CallingConv::ID CC = F.getCallingConv(); + if (!AMDGPU::isEntryFunctionCC(CC)) { + A.getOrCreateAAFor<AAAMDFlatWorkGroupSize>(IRPosition::function(F)); + A.getOrCreateAAFor<AAAMDWavesPerEU>(IRPosition::function(F)); + } else if (CC == CallingConv::AMDGPU_KERNEL) { + addPreloadKernArgHint(F, TM); } } `````````` </details> https://github.com/llvm/llvm-project/pull/101591 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
