Skip to content

Commit 5f9d3d7

Browse files
committed
C++: Rename 'ScrutineeValueNumber' to 'SwitchConditionValueNumber'.
1 parent feea29b commit 5f9d3d7

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

cpp/ql/lib/semmle/code/cpp/controlflow/IRGuards.qll

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -109,11 +109,11 @@ private class CompareGEValueNumber extends CompareValueNumber {
109109
* A value number such that at least one of the instructions provides
110110
* the integer value controlling a `SwitchInstruction`.
111111
*/
112-
private class ScrutineeValueNumber extends ValueNumber {
112+
private class SwitchConditionValueNumber extends ValueNumber {
113113
SwitchInstruction switch;
114114

115115
pragma[nomagic]
116-
ScrutineeValueNumber() { this.getAnInstruction() = switch.getExpression() }
116+
SwitchConditionValueNumber() { this.getAnInstruction() = switch.getExpression() }
117117

118118
/** Gets an expression that belongs to this value number. */
119119
Operand getExpressionOperand() { result = switch.getExpressionOperand() }
@@ -990,11 +990,11 @@ private predicate isRelevantUnaryComparisonOperand(Operand op) {
990990
private predicate unary_simple_comparison_eq(
991991
ValueNumber test, Operand op, int k, boolean inNonZeroCase, AbstractValue value
992992
) {
993-
exists(CaseEdge case, ScrutineeValueNumber scrutinee |
994-
scrutinee = test and
995-
op = scrutinee.getExpressionOperand() and
993+
exists(CaseEdge case, SwitchConditionValueNumber condition |
994+
condition = test and
995+
op = condition.getExpressionOperand() and
996996
case = value.(MatchValue).getCase() and
997-
exists(scrutinee.getSuccessor(case)) and
997+
exists(condition.getSuccessor(case)) and
998998
case.getValue().toInt() = k and
999999
inNonZeroCase = false
10001000
)
@@ -1158,7 +1158,7 @@ private predicate simple_comparison_lt(CompareValueNumber cmp, Operand left, Ope
11581158

11591159
/** Rearrange various simple comparisons into `op < k` form. */
11601160
private predicate unary_simple_comparison_lt(
1161-
ScrutineeValueNumber test, Operand op, int k, boolean isLt, AbstractValue value
1161+
SwitchConditionValueNumber test, Operand op, int k, boolean isLt, AbstractValue value
11621162
) {
11631163
exists(CaseEdge case |
11641164
test.getExpressionOperand() = op and

0 commit comments

Comments
 (0)