Skip to content

Commit 498bfd2

Browse files
authored
Merge pull request #18512 from aschackmull/dataflow/join-fix2
Dataflow: Another minor join-order fix
2 parents 65a11e4 + 20af2d5 commit 498bfd2

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

shared/dataflow/codeql/dataflow/internal/DataFlowImpl.qll

+3-2
Original file line numberDiff line numberDiff line change
@@ -2154,8 +2154,7 @@ module MakeImpl<LocationSig Location, InputSig<Location> Lang> {
21542154
pragma[nomagic]
21552155
private predicate storeStepFwd(NodeEx node1, Ap ap1, Content c, NodeEx node2, Ap ap2) {
21562156
fwdFlowStore(node1, _, ap1, _, c, _, _, node2, _, _, _) and
2157-
ap2 = apCons(c, ap1) and
2158-
readStepFwd(_, ap2, c, _, _)
2157+
readStepFwd(_, ap2, c, _, ap1)
21592158
}
21602159

21612160
pragma[nomagic]
@@ -4393,6 +4392,7 @@ module MakeImpl<LocationSig Location, InputSig<Location> Lang> {
43934392
Typ getTyp(DataFlowType t) { result = t }
43944393

43954394
bindingset[c, tail]
4395+
pragma[inline_late]
43964396
Ap apCons(Content c, Ap tail) { result.isCons(c, tail) }
43974397

43984398
class ApHeadContent = Content;
@@ -4462,6 +4462,7 @@ module MakeImpl<LocationSig Location, InputSig<Location> Lang> {
44624462
abstract Content getHead();
44634463

44644464
/** Holds if this is a representation of `head` followed by `tail`. */
4465+
pragma[nomagic]
44654466
abstract predicate isCons(Content head, AccessPath tail);
44664467

44674468
/** Gets the front of this access path. */

0 commit comments

Comments
 (0)