Skip to content

Commit 27e6173

Browse files
committed
C++: Remove SemSsaExplicitUpdate.getSourceExpr.
1 parent 3a73faf commit 27e6173

File tree

3 files changed

+2
-4
lines changed

3 files changed

+2
-4
lines changed

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,6 @@ class SemSsaExplicitUpdate extends SemSsaVariable {
2222

2323
SemSsaExplicitUpdate() { Specific::explicitUpdate(this, sourceExpr) }
2424

25-
final SemExpr getSourceExpr() { result = sourceExpr }
26-
2725
final SemExpr getDefiningExpr() { result = sourceExpr }
2826
}
2927

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ private predicate constantIntegerExpr(SemExpr e, int val) {
1414
// Copy of another constant
1515
exists(SemSsaExplicitUpdate v, SemExpr src |
1616
e = v.getAUse() and
17-
src = v.getSourceExpr() and
17+
src = v.getDefiningExpr() and
1818
constantIntegerExpr(src, val)
1919
)
2020
or

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ module SignAnalysis<DeltaSig D, UtilSig<Sem, D> Utils> {
3939

4040
/** An SSA definition whose sign is determined by the sign of that definitions source expression. */
4141
private class ExplicitSignDef extends FlowSignDef instanceof SemSsaExplicitUpdate {
42-
final override Sign getSign() { result = semExprSign(super.getSourceExpr()) }
42+
final override Sign getSign() { result = semExprSign(super.getDefiningExpr()) }
4343
}
4444

4545
/** An SSA Phi definition, whose sign is the union of the signs of its inputs. */

0 commit comments

Comments
 (0)