From 2a2733d18c42924360739e97b1146b70087b3695 Mon Sep 17 00:00:00 2001 From: Anders Schack-Mulligen Date: Wed, 13 Sep 2023 13:10:15 +0200 Subject: [PATCH] Dataflow: One more minor perf tweak. --- shared/dataflow/codeql/dataflow/internal/DataFlowImpl.qll | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/shared/dataflow/codeql/dataflow/internal/DataFlowImpl.qll b/shared/dataflow/codeql/dataflow/internal/DataFlowImpl.qll index 4460bba4646ec..7d9ef3e2404e5 100644 --- a/shared/dataflow/codeql/dataflow/internal/DataFlowImpl.qll +++ b/shared/dataflow/codeql/dataflow/internal/DataFlowImpl.qll @@ -1517,7 +1517,7 @@ module MakeImpl { pragma[nomagic] private predicate fwdFlowOutCand( - DataFlowCall call, DataFlowCallable inner, NodeEx out, FlowState state, CcNoCall outercc, + DataFlowCall call, DataFlowCallable inner, NodeEx out, FlowState state, ParamNodeOption summaryCtx, TypOption argT, ApOption argAp, Typ t, Ap ap, ApApprox apa ) { exists(RetNodeEx ret, boolean allowsFieldFlow, CcNoCall innercc | @@ -1531,7 +1531,6 @@ module MakeImpl { innercc) ) | - outercc = getCallContextReturn(inner, call) and if allowsFieldFlow = false then ap instanceof ApNil else any() ) } @@ -1540,8 +1539,9 @@ module MakeImpl { DataFlowCall call, DataFlowCallable inner, NodeEx out, FlowState state, CcNoCall outercc, ParamNodeOption summaryCtx, TypOption argT, ApOption argAp, Typ t, Ap ap, ApApprox apa ) { - fwdFlowOutCand(call, inner, out, state, outercc, summaryCtx, argT, argAp, t, ap, apa) and - FwdTypeFlow::typeFlowValidEdgeOut(call, inner) + fwdFlowOutCand(call, inner, out, state, summaryCtx, argT, argAp, t, ap, apa) and + FwdTypeFlow::typeFlowValidEdgeOut(call, inner) and + outercc = getCallContextReturn(inner, call) } private module FwdTypeFlowInput implements TypeFlowInput {