Skip to content

Commit 048a7c4

Browse files
committed
Rangeanalysis: Rename SsaBound.getAVariable to getVariable.
1 parent 48291dd commit 048a7c4

File tree

4 files changed

+9
-9
lines changed

4 files changed

+9
-9
lines changed

cpp/ql/lib/semmle/code/cpp/rangeanalysis/new/internal/semantic/analysis/ModulusAnalysis.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ module ModulusAnalysis<DeltaSig D, BoundSig<SemLocation, Sem, D> Bounds, UtilSig
151151
) {
152152
exists(Bounds::SemSsaBound phibound, int v, int m |
153153
edge.phiInput(phi, inp) and
154-
phibound.getAVariable() = phi and
154+
phibound.getVariable() = phi and
155155
ssaModulus(inp, edge, phibound, v, m) and
156156
mod = m.gcd(v) and
157157
mod != 1
@@ -233,7 +233,7 @@ module ModulusAnalysis<DeltaSig D, BoundSig<SemLocation, Sem, D> Bounds, UtilSig
233233
) {
234234
phiModulus(v, b, val, mod) and pos.hasReadOfVar(v)
235235
or
236-
b.(Bounds::SemSsaBound).getAVariable() = v and pos.hasReadOfVar(v) and val = 0 and mod = 0
236+
b.(Bounds::SemSsaBound).getVariable() = v and pos.hasReadOfVar(v) and val = 0 and mod = 0
237237
or
238238
exists(SemExpr e, int val0, int delta |
239239
semExprModulus(e, b, val0, mod) and

cpp/ql/lib/semmle/code/cpp/rangeanalysis/new/internal/semantic/analysis/RangeAnalysisImpl.qll

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ module ConstantBounds implements BoundSig<SemLocation, Sem, FloatDelta> {
128128
class SemZeroBound extends SemBound instanceof SemanticBound::SemZeroBound { }
129129

130130
class SemSsaBound extends SemBound instanceof SemanticBound::SemSsaBound {
131-
SemSsaVariable getAVariable() { result = this.(SemanticBound::SemSsaBound).getAVariable() }
131+
SemSsaVariable getVariable() { result = this.(SemanticBound::SemSsaBound).getAVariable() }
132132
}
133133
}
134134

@@ -146,7 +146,7 @@ module RelativeBounds implements BoundSig<SemLocation, Sem, FloatDelta> {
146146
class SemZeroBound extends SemBound instanceof SemanticBound::SemZeroBound { }
147147

148148
class SemSsaBound extends SemBound instanceof SemanticBound::SemSsaBound {
149-
SemSsaVariable getAVariable() { result = this.(SemanticBound::SemSsaBound).getAVariable() }
149+
SemSsaVariable getVariable() { result = this.(SemanticBound::SemSsaBound).getAVariable() }
150150
}
151151
}
152152

@@ -162,7 +162,7 @@ module AllBounds implements BoundSig<SemLocation, Sem, FloatDelta> {
162162
class SemZeroBound extends SemBound instanceof SemanticBound::SemZeroBound { }
163163

164164
class SemSsaBound extends SemBound instanceof SemanticBound::SemSsaBound {
165-
SemSsaVariable getAVariable() { result = this.(SemanticBound::SemSsaBound).getAVariable() }
165+
SemSsaVariable getVariable() { result = this.(SemanticBound::SemSsaBound).getAVariable() }
166166
}
167167
}
168168

java/ql/lib/semmle/code/java/dataflow/RangeAnalysis.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -419,7 +419,7 @@ module Bounds implements BoundSig<Location, Sem, IntDelta> {
419419
class SemZeroBound = ZeroBound;
420420

421421
class SemSsaBound extends SsaBound {
422-
Sem::SsaVariable getAVariable() { result = super.getSsa() }
422+
Sem::SsaVariable getVariable() { result = super.getSsa() }
423423
}
424424
}
425425

shared/rangeanalysis/codeql/rangeanalysis/RangeAnalysis.qll

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -345,7 +345,7 @@ signature module BoundSig<LocationSig Location, Semantic Sem, DeltaSig D> {
345345
class SemZeroBound extends SemBound;
346346

347347
class SemSsaBound extends SemBound {
348-
Sem::SsaVariable getAVariable();
348+
Sem::SsaVariable getVariable();
349349
}
350350
}
351351

@@ -917,7 +917,7 @@ module RangeStage<
917917
or
918918
boundedPhi(inp, b, d, upper, fromBackEdge0, origdelta, reason)
919919
or
920-
b.(SemSsaBound).getAVariable() = inp and
920+
b.(SemSsaBound).getVariable() = inp and
921921
d = D::fromFloat(0) and
922922
(upper = true or upper = false) and
923923
fromBackEdge0 = false and
@@ -967,7 +967,7 @@ module RangeStage<
967967
Sem::SsaPhiNode phi, Sem::SsaVariable inp, Sem::SsaReadPositionPhiInputEdge edge, boolean upper
968968
) {
969969
exists(D::Delta d, SemSsaBound phibound |
970-
phibound.getAVariable() = phi and
970+
phibound.getVariable() = phi and
971971
boundedPhiInp(phi, inp, edge, phibound, d, upper, _, _, _) and
972972
(
973973
upper = true and D::toFloat(d) <= 0

0 commit comments

Comments
 (0)