[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