[llvm] [llvm][amdgpu] Handle indirect refs to LDS GVs during LDS lowering (PR #124089)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 23 01:15:13 PST 2025


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 2a51a0d39a659feeeee57b6d1d768bf08d378c5e ae2e16be81fca9791ea7c3ef74304fa98a3b9661 --extensions cpp -- llvm/lib/Target/AMDGPU/AMDGPUMemoryUtils.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUMemoryUtils.cpp b/llvm/lib/Target/AMDGPU/AMDGPUMemoryUtils.cpp
index fc223e2390..a5bfdb7bf6 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUMemoryUtils.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUMemoryUtils.cpp
@@ -142,7 +142,7 @@ LDSUsesInfoTy getTransitiveUsesOfLDS(const CallGraph &CG, Module &M) {
   getUsesOfLDSByFunction(CG, M, DirectMapKernel, DirectMapFunction);
 
   // Collect functions whose address has escaped
-  DenseSet<Function*> AddressTakenFuncs;
+  DenseSet<Function *> AddressTakenFuncs;
   for (Function &F : M.functions()) {
     if (!isKernelLDS(&F))
       if (F.hasAddressTaken(nullptr,
@@ -150,7 +150,7 @@ LDSUsesInfoTy getTransitiveUsesOfLDS(const CallGraph &CG, Module &M) {
                             /* IgnoreAssumeLikeCalls */ false,
                             /* IgnoreLLVMUsed */ true,
                             /* IgnoreArcAttachedCall */ false)) {
-          AddressTakenFuncs.insert(&F);
+        AddressTakenFuncs.insert(&F);
       }
   }
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/124089


More information about the llvm-commits mailing list