[NFC][AMDGPU] Reformat code for creating AA

[shiltian]

No description provided.

[shiltian]

• [AMDGPU] Enable AAAddressSpace in AMDGPUAttributor #101593
• [NFC][AMDGPU] Reformat code for creating AA #101591 👈
• [Attributor] Indicate optimistic fixed point if an instruction already has non-zero address space #101589
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @shiltian and the rest of your teammates on Graphite

[llvmbot]

@llvm/pr-subscribers-backend-amdgpu

Author: Shilei Tian (shiltian)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/101591.diff

1 Files Affected:

• (modified) llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp (+12-11)

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);
     }
   }