inference: infer_compilation_signatures for even more cases (#56552)

Refs #56495
JuliaLang · Nov 14, 2024 · 286ece2 · 286ece2
1 parent aa05c98
commit 286ece2
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/Compiler/src/abstractinterpretation.jl b/Compiler/src/abstractinterpretation.jl
@@ -207,14 +207,14 @@ function abstract_call_gf_by_type(interp::AbstractInterpreter, @nospecialize(f),
         rettype = from_interprocedural!(interp, rettype, sv, arginfo, conditionals)
 
         # Also considering inferring the compilation signature for this method, so
-        # it is available to the compiler in case it ends up needing it for the invoke.
+        # it is available to the compiler, unless it should not end up needing it (for an invoke).
         if (isa(sv, InferenceState) && infer_compilation_signature(interp) &&
-            (seenall && 1 == napplicable) && !is_removable_if_unused(all_effects))
+            (seenall && 1 == napplicable) && (!is_removable_if_unused(all_effects) || !call_result_unused(si)))
             (; match) = applicable[1]
             method = match.method
             sig = match.spec_types
             mi = specialize_method(match; preexisting=true)
-            if mi !== nothing && !const_prop_methodinstance_heuristic(interp, mi, arginfo, sv)
+            if mi === nothing || !const_prop_methodinstance_heuristic(interp, mi, arginfo, sv)
                 csig = get_compileable_sig(method, sig, match.sparams)
                 if csig !== nothing && csig !== sig
                     abstract_call_method(interp, method, csig, match.sparams, multiple_matches, StmtInfo(false), sv)::Future