[llvm] Check for Res first before checking if we are to export it (PR #152346)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 6 10:34:41 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-transforms
Author: AZero13 (AZero13)
<details>
<summary>Changes</summary>
Res can be null, though it is unclear if res is always set if the cs info is exported. Better to be safe just in case.
---
Full diff: https://github.com/llvm/llvm-project/pull/152346.diff
1 Files Affected:
- (modified) llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp (+1-1)
``````````diff
diff --git a/llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp b/llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp
index aec484f8a18f9..e20025a997b8c 100644
--- a/llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp
+++ b/llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp
@@ -1648,7 +1648,7 @@ bool DevirtModule::tryUniformRetValOpt(
if (Target.RetVal != TheRetVal)
return false;
- if (CSInfo.isExported()) {
+ if (Res && CSInfo.isExported()) {
Res->TheKind = WholeProgramDevirtResolution::ByArg::UniformRetVal;
Res->Info = TheRetVal;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/152346
More information about the llvm-commits
mailing list