[llvm] [AMDGPU][SplitModule] Keep looking for more dependencies after finding an indirect call (PR #93480)
via llvm-commits
llvm-commits at lists.llvm.org
Mon May 27 07:24:48 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-amdgpu
Author: Pierre van Houtryve (Pierre-vh)
<details>
<summary>Changes</summary>
This is just something I noticed while going over this pass logic one more time and didn't cause issues (yet). If we find an indirect call, we stop looking assuming we added all functions to the list, but if not all functions in the module were indirectly callable, some may still be missing.
Just to be safe, keep looking until we did everything we could to find dependencies, so we don't accidentally miss one.
---
Full diff: https://github.com/llvm/llvm-project/pull/93480.diff
1 Files Affected:
- (modified) llvm/lib/Target/AMDGPU/AMDGPUSplitModule.cpp (+1-1)
``````````diff
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUSplitModule.cpp b/llvm/lib/Target/AMDGPU/AMDGPUSplitModule.cpp
index dab773f28752d..2449fa581842a 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUSplitModule.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUSplitModule.cpp
@@ -340,7 +340,7 @@ static void addAllDependencies(SplitModuleLogger &SML, const CallGraph &CG,
// TODO: Print an ORE as well ?
addAllIndirectCallDependencies(M, Fns);
HadIndirectCall = true;
- return;
+ continue;
}
if (Callee->isDeclaration())
``````````
</details>
https://github.com/llvm/llvm-project/pull/93480
More information about the llvm-commits
mailing list