Skip to content

Commit 8e458f4

Browse files
committed
C++: Simplify local variable names
1 parent f9bfd96 commit 8e458f4

File tree

4 files changed

+20
-20
lines changed

4 files changed

+20
-20
lines changed

cpp/ql/test/library-tests/ir/ir/PrintAST.expected

+4-4
Original file line numberDiff line numberDiff line change
@@ -24100,19 +24100,19 @@ ir.cpp:
2410024100
# 2721| Type = [IntType] int
2410124101
# 2724| [TopLevelFunction] char UseBracketOperator(WithBracketOperator const, int)
2410224102
# 2724| <params>:
24103-
# 2724| getParameter(0): [Parameter] line
24103+
# 2724| getParameter(0): [Parameter] x
2410424104
# 2724| Type = [SpecifiedType] const WithBracketOperator
24105-
# 2724| getParameter(1): [Parameter] indent
24105+
# 2724| getParameter(1): [Parameter] i
2410624106
# 2724| Type = [IntType] int
2410724107
# 2724| getEntryPoint(): [BlockStmt] { ... }
2410824108
# 2725| getStmt(0): [ReturnStmt] return ...
2410924109
# 2725| getExpr(): [OverloadedArrayExpr] call to operator[]
2411024110
# 2725| Type = [LValueReferenceType] const char &
2411124111
# 2725| ValueCategory = prvalue
24112-
# 2725| getArrayBase(): [VariableAccess] line
24112+
# 2725| getArrayBase(): [VariableAccess] x
2411324113
# 2725| Type = [SpecifiedType] const WithBracketOperator
2411424114
# 2725| ValueCategory = lvalue
24115-
# 2725| getArrayOffset(): [VariableAccess] indent
24115+
# 2725| getArrayOffset(): [VariableAccess] i
2411624116
# 2725| Type = [IntType] int
2411724117
# 2725| ValueCategory = prvalue(load)
2411824118
# 2725| getExpr().getFullyConverted(): [ReferenceDereferenceExpr] (reference dereference)

cpp/ql/test/library-tests/ir/ir/aliased_ir.expected

+7-7
Original file line numberDiff line numberDiff line change
@@ -19719,16 +19719,16 @@ ir.cpp:
1971919719
# 2724| m2724_2(unknown) = AliasedDefinition :
1972019720
# 2724| m2724_3(unknown) = InitializeNonLocal :
1972119721
# 2724| m2724_4(unknown) = Chi : total:m2724_2, partial:m2724_3
19722-
# 2724| r2724_5(glval<WithBracketOperator>) = VariableAddress[line] :
19723-
# 2724| m2724_6(WithBracketOperator) = InitializeParameter[line] : &:r2724_5
19722+
# 2724| r2724_5(glval<WithBracketOperator>) = VariableAddress[x] :
19723+
# 2724| m2724_6(WithBracketOperator) = InitializeParameter[x] : &:r2724_5
1972419724
# 2724| m2724_7(unknown) = Chi : total:m2724_4, partial:m2724_6
19725-
# 2724| r2724_8(glval<int>) = VariableAddress[indent] :
19726-
# 2724| m2724_9(int) = InitializeParameter[indent] : &:r2724_8
19725+
# 2724| r2724_8(glval<int>) = VariableAddress[i] :
19726+
# 2724| m2724_9(int) = InitializeParameter[i] : &:r2724_8
1972719727
# 2725| r2725_1(glval<char>) = VariableAddress[#return] :
19728-
# 2725| r2725_2(glval<WithBracketOperator>) = VariableAddress[line] :
19728+
# 2725| r2725_2(glval<WithBracketOperator>) = VariableAddress[x] :
1972919729
# 2725| r2725_3(glval<unknown>) = FunctionAddress[operator[]] :
19730-
# 2725| r2725_4(glval<int>) = VariableAddress[indent] :
19731-
# 2725| r2725_5(int) = Load[indent] : &:r2725_4, m2724_9
19730+
# 2725| r2725_4(glval<int>) = VariableAddress[i] :
19731+
# 2725| r2725_5(int) = Load[i] : &:r2725_4, m2724_9
1973219732
# 2725| r2725_6(char &) = Call[operator[]] : func:r2725_3, this:r2725_2, 0:r2725_5
1973319733
# 2725| m2725_7(unknown) = ^CallSideEffect : ~m2724_7
1973419734
# 2725| m2725_8(unknown) = Chi : total:m2724_7, partial:m2725_7

cpp/ql/test/library-tests/ir/ir/ir.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -2721,8 +2721,8 @@ struct WithBracketOperator {
27212721
const char& operator[](int pos) const;
27222722
};
27232723

2724-
char UseBracketOperator(const WithBracketOperator line, int indent) {
2725-
return line[indent];
2724+
char UseBracketOperator(const WithBracketOperator x, int i) {
2725+
return x[i];
27262726
}
27272727

27282728
// semmle-extractor-options: -std=c++20 --clang

cpp/ql/test/library-tests/ir/ir/raw_ir.expected

+7-7
Original file line numberDiff line numberDiff line change
@@ -18036,15 +18036,15 @@ ir.cpp:
1803618036
# 2724| v2724_1(void) = EnterFunction :
1803718037
# 2724| mu2724_2(unknown) = AliasedDefinition :
1803818038
# 2724| mu2724_3(unknown) = InitializeNonLocal :
18039-
# 2724| r2724_4(glval<WithBracketOperator>) = VariableAddress[line] :
18040-
# 2724| mu2724_5(WithBracketOperator) = InitializeParameter[line] : &:r2724_4
18041-
# 2724| r2724_6(glval<int>) = VariableAddress[indent] :
18042-
# 2724| mu2724_7(int) = InitializeParameter[indent] : &:r2724_6
18039+
# 2724| r2724_4(glval<WithBracketOperator>) = VariableAddress[x] :
18040+
# 2724| mu2724_5(WithBracketOperator) = InitializeParameter[x] : &:r2724_4
18041+
# 2724| r2724_6(glval<int>) = VariableAddress[i] :
18042+
# 2724| mu2724_7(int) = InitializeParameter[i] : &:r2724_6
1804318043
# 2725| r2725_1(glval<char>) = VariableAddress[#return] :
18044-
# 2725| r2725_2(glval<WithBracketOperator>) = VariableAddress[line] :
18044+
# 2725| r2725_2(glval<WithBracketOperator>) = VariableAddress[x] :
1804518045
# 2725| r2725_3(glval<unknown>) = FunctionAddress[operator[]] :
18046-
# 2725| r2725_4(glval<int>) = VariableAddress[indent] :
18047-
# 2725| r2725_5(int) = Load[indent] : &:r2725_4, ~m?
18046+
# 2725| r2725_4(glval<int>) = VariableAddress[i] :
18047+
# 2725| r2725_5(int) = Load[i] : &:r2725_4, ~m?
1804818048
# 2725| r2725_6(char &) = Call[operator[]] : func:r2725_3, this:r2725_2, 0:r2725_5
1804918049
# 2725| mu2725_7(unknown) = ^CallSideEffect : ~m?
1805018050
# 2725| v2725_8(void) = ^IndirectReadSideEffect[-1] : &:r2725_2, ~m?

0 commit comments

Comments
 (0)