Skip to content

Commit 2850b8e

Browse files
committed
JS: Fix RangeAnalysis after BasicBlock.dominates change
1 parent cbfa5ad commit 2850b8e

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

javascript/ql/src/semmle/javascript/RangeAnalysis.qll

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -606,10 +606,10 @@ module RangeAnalysis {
606606
cfg2BB = cfg2.getBasicBlock() and
607607
cfg2RBB = cfg2BB.(ReachableBasicBlock) and
608608
(
609-
cfg1RBB.strictlyDominates(cfg2BB) and
609+
cfg2BB.getImmediateDominator+() = cfg1RBB and
610610
cfg = cfg2
611611
or
612-
cfg2RBB.strictlyDominates(cfg1RBB) and
612+
cfg1BB.getImmediateDominator+() = cfg2BB and
613613
cfg = cfg1
614614
)
615615
)
@@ -681,7 +681,7 @@ module RangeAnalysis {
681681
midBB = midcfg.getBasicBlock() and
682682
midRBB = midBB.(ReachableBasicBlock) and
683683
cfgBB = cfg.getBasicBlock() and
684-
midRBB.strictlyDominates(cfgBB)
684+
cfgBB.getImmediateDominator+() = midRBB
685685
)
686686
}
687687

0 commit comments

Comments
 (0)