Skip to content

Commit e7580b6

Browse files
authored
Merge pull request #8852 from jketema/frontend-update
Fix test regressions due to C++ frontend update
2 parents 8d2bd66 + 73e4f0c commit e7580b6

File tree

25 files changed

+845
-817
lines changed

25 files changed

+845
-817
lines changed

cpp/ql/test/library-tests/controlflow/dominance/bbIDominates.expected

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,4 +31,4 @@
3131
| test2 | 83 | if (...) ... | 74 | test2 |
3232
| test2 | 83 | if (...) ... | 84 | break; |
3333
| test2 | 83 | if (...) ... | 85 | if (...) ... |
34-
| test2 | 85 | if (...) ... | 0 | { ... } |
34+
| test2 | 85 | if (...) ... | 86 | { ... } |

cpp/ql/test/library-tests/controlflow/dominance/dominator.expected

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@
118118
| test | 44 | ... = ... | 42 | j |
119119
| test | 44 | ExprStmt | 44 | 10 |
120120
| test | 44 | w | 44 | ... = ... |
121-
| test | 47 | ... += ... | 50 | for(...;...;...) ... |
121+
| test | 47 | ... += ... | 50 | { ... } |
122122
| test | 47 | ExprStmt | 47 | z |
123123
| test | 47 | w | 47 | ... += ... |
124124
| test | 47 | z | 47 | w |
@@ -133,6 +133,7 @@
133133
| test | 50 | j | 50 | ... ++ |
134134
| test | 50 | j | 50 | ... = ... |
135135
| test | 50 | label ...: | 50 | j |
136+
| test | 50 | { ... } | 50 | for(...;...;...) ... |
136137
| test | 50 | { ... } | 51 | ExprStmt |
137138
| test | 51 | 30 | 51 | y |
138139
| test | 51 | ... = ... | 52 | if (...) ... |
@@ -181,10 +182,10 @@
181182
| test | 70 | ExprStmt | 70 | 40 |
182183
| test | 70 | w | 70 | ... = ... |
183184
| test | 71 | return ... | 71 | w |
184-
| test2 | 0 | { ... } | 86 | ExprStmt |
185185
| test2 | 74 | { ... } | 76 | declaration |
186-
| test2 | 76 | declaration | 77 | for(...;...;...) ... |
186+
| test2 | 76 | declaration | 77 | { ... } |
187187
| test2 | 77 | for(...;...;...) ... | 77 | { ... } |
188+
| test2 | 77 | { ... } | 77 | for(...;...;...) ... |
188189
| test2 | 77 | { ... } | 78 | ExprStmt |
189190
| test2 | 78 | 10 | 78 | b |
190191
| test2 | 78 | ... = ... | 79 | if (...) ... |
@@ -211,11 +212,12 @@
211212
| test2 | 83 | if (...) ... | 83 | a |
212213
| test2 | 84 | break; | 87 | label ...: |
213214
| test2 | 85 | 20 | 85 | ... == ... |
214-
| test2 | 85 | ... == ... | 0 | { ... } |
215+
| test2 | 85 | ... == ... | 86 | { ... } |
215216
| test2 | 85 | a | 85 | 20 |
216217
| test2 | 85 | if (...) ... | 85 | a |
217218
| test2 | 86 | ExprStmt | 86 | c |
218219
| test2 | 86 | c | 86 | return ... |
220+
| test2 | 86 | { ... } | 86 | ExprStmt |
219221
| test2 | 87 | label ...: | 88 | ExprStmt |
220222
| test2 | 88 | ExprStmt | 88 | b |
221223
| test2 | 88 | b | 88 | return ... |

cpp/ql/test/library-tests/dataflow/dataflow-tests/dataflow-ir-consistency.expected

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,10 +218,10 @@ postWithInFlow
218218
| lambdas.cpp:20:11:20:11 | FieldAddress [post update] | PostUpdateNode should not be the target of local flow. |
219219
| lambdas.cpp:20:11:20:11 | FieldAddress [post update] | PostUpdateNode should not be the target of local flow. |
220220
| lambdas.cpp:20:11:20:11 | FieldAddress [post update] | PostUpdateNode should not be the target of local flow. |
221-
| lambdas.cpp:23:3:23:3 | (reference dereference) [post update] | PostUpdateNode should not be the target of local flow. |
222221
| lambdas.cpp:23:3:23:14 | FieldAddress [post update] | PostUpdateNode should not be the target of local flow. |
223222
| lambdas.cpp:23:3:23:14 | VariableAddress [post update] | PostUpdateNode should not be the target of local flow. |
224223
| lambdas.cpp:23:3:23:14 | v [post update] | PostUpdateNode should not be the target of local flow. |
224+
| lambdas.cpp:23:15:23:15 | (reference dereference) [post update] | PostUpdateNode should not be the target of local flow. |
225225
| lambdas.cpp:28:7:28:7 | VariableAddress [post update] | PostUpdateNode should not be the target of local flow. |
226226
| lambdas.cpp:28:10:31:2 | FieldAddress [post update] | PostUpdateNode should not be the target of local flow. |
227227
| lambdas.cpp:28:10:31:2 | FieldAddress [post update] | PostUpdateNode should not be the target of local flow. |

cpp/ql/test/library-tests/depends_initializers/InitializerAccesses.expected

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
| initializers.cpp:18:8:18:15 | initializer for y | initializers.cpp:18:9:18:9 | x |
22
| initializers.cpp:19:8:19:21 | initializer for z | initializers.cpp:19:9:19:9 | x |
33
| initializers.cpp:23:8:23:9 | initializer for j | initializers.cpp:23:9:23:9 | i |
4-
| initializers.cpp:27:8:27:10 | initializer for aax | initializers.cpp:27:9:27:10 | ax |
4+
| initializers.cpp:27:9:27:10 | initializer for aax | initializers.cpp:27:9:27:10 | ax |
55
| initializers.cpp:30:20:30:35 | initializer for myIntArray | initializers.cpp:30:22:30:22 | i |
66
| template_static.cpp:9:24:9:24 | initializer for static_c | template_static.cpp:9:24:9:24 | c |
77
| template_static.cpp:10:22:10:24 | initializer for static_v | template_static.cpp:10:24:10:24 | v |

cpp/ql/test/library-tests/depends_initializers/InitializerCFG.expected

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
| initializers.cpp:23:8:23:9 | initializer for j | |
99
| initializers.cpp:25:3:25:3 | initializer for a | |
1010
| initializers.cpp:26:7:26:10 | initializer for ax | |
11-
| initializers.cpp:27:8:27:10 | initializer for aax | |
11+
| initializers.cpp:27:9:27:10 | initializer for aax | |
1212
| initializers.cpp:30:20:30:35 | initializer for myIntArray | |
1313
| initializers.cpp:31:3:31:15 | initializer for myObjectArray | |
1414
| template_static.cpp:2:15:2:15 | initializer for c | |
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
| file://:0:0:0:0 | There was an error during this compilation |
22
| float128.cpp:1:39:1:39 | 128-bit floating-point types are not supported in this configuration |
3-
| float128.cpp:2:30:2:30 | 128-bit floating-point types are not supported in this configuration |
3+
| float128.cpp:2:30:2:30 | an attribute specifies a mode incompatible with '<error-type>' |
4+
| float128.cpp:2:41:2:41 | invalid combination of type specifiers |

cpp/ql/test/library-tests/floats/float128/usertypes.expected

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
| float128.cpp:1:50:1:60 | _Complex128 | file://:0:0:0:0 | <error-type> |
2-
| float128.cpp:2:41:2:49 | _Float128 | file://:0:0:0:0 | <error-type> |
32
| float128.cpp:13:29:13:54 | __is_floating_point_helper<T> | float128.cpp:10:8:10:17 | false_type |
43
| float128.cpp:14:19:14:51 | __is_floating_point_helper<float> | float128.cpp:11:8:11:16 | true_type |
54
| float128.cpp:15:19:15:52 | __is_floating_point_helper<double> | float128.cpp:11:8:11:16 | true_type |

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

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -4833,9 +4833,6 @@
48334833
| ir.cpp:1043:24:1043:24 | SideEffect | ~m1043_20 |
48344834
| ir.cpp:1043:31:1043:31 | Address | &:r1043_9 |
48354835
| ir.cpp:1043:36:1043:55 | Address | &:r1043_11 |
4836-
| ir.cpp:1043:43:1043:43 | Address | &:r1043_16 |
4837-
| ir.cpp:1043:43:1043:43 | Arg(this) | this:r1043_16 |
4838-
| ir.cpp:1043:43:1043:43 | SideEffect | ~m1043_20 |
48394836
| ir.cpp:1043:43:1043:54 | Address | &:r1043_22 |
48404837
| ir.cpp:1043:43:1043:54 | Address | &:r1043_24 |
48414838
| ir.cpp:1043:43:1043:54 | Address | &:r1043_25 |
@@ -4856,8 +4853,11 @@
48564853
| ir.cpp:1043:45:1043:49 | SideEffect | ~m1043_4 |
48574854
| ir.cpp:1043:45:1043:49 | Unary | r1043_13 |
48584855
| ir.cpp:1043:45:1043:49 | Unary | r1043_15 |
4859-
| ir.cpp:1043:53:1043:53 | Load | ~m1043_20 |
4860-
| ir.cpp:1043:53:1043:53 | Right | r1043_26 |
4856+
| ir.cpp:1043:52:1043:52 | Address | &:r1043_16 |
4857+
| ir.cpp:1043:52:1043:52 | Arg(this) | this:r1043_16 |
4858+
| ir.cpp:1043:52:1043:52 | SideEffect | ~m1043_20 |
4859+
| ir.cpp:1043:54:1043:54 | Load | ~m1043_20 |
4860+
| ir.cpp:1043:54:1043:54 | Right | r1043_26 |
48614861
| ir.cpp:1043:58:1043:58 | ChiPartial | partial:m1043_9 |
48624862
| ir.cpp:1043:58:1043:58 | ChiTotal | total:m1043_3 |
48634863
| ir.cpp:1043:58:1043:58 | StoreValue | r1043_8 |
@@ -4972,9 +4972,6 @@
49724972
| ir.cpp:1047:34:1047:34 | SideEffect | ~m1047_20 |
49734973
| ir.cpp:1047:41:1047:41 | Address | &:r1047_9 |
49744974
| ir.cpp:1047:46:1047:65 | Address | &:r1047_11 |
4975-
| ir.cpp:1047:53:1047:53 | Address | &:r1047_16 |
4976-
| ir.cpp:1047:53:1047:53 | Arg(this) | this:r1047_16 |
4977-
| ir.cpp:1047:53:1047:53 | SideEffect | ~m1047_20 |
49784975
| ir.cpp:1047:53:1047:64 | Address | &:r1047_23 |
49794976
| ir.cpp:1047:53:1047:64 | Load | ~m1047_20 |
49804977
| ir.cpp:1047:53:1047:64 | StoreValue | r1047_24 |
@@ -4989,6 +4986,9 @@
49894986
| ir.cpp:1047:55:1047:59 | SideEffect | ~m1047_4 |
49904987
| ir.cpp:1047:55:1047:59 | Unary | r1047_13 |
49914988
| ir.cpp:1047:55:1047:59 | Unary | r1047_15 |
4989+
| ir.cpp:1047:62:1047:62 | Address | &:r1047_16 |
4990+
| ir.cpp:1047:62:1047:62 | Arg(this) | this:r1047_16 |
4991+
| ir.cpp:1047:62:1047:62 | SideEffect | ~m1047_20 |
49924992
| ir.cpp:1047:63:1047:63 | Right | r1047_22 |
49934993
| ir.cpp:1047:68:1047:68 | StoreValue | r1047_8 |
49944994
| ir.cpp:1047:68:1047:68 | Unary | r1047_7 |
@@ -5097,9 +5097,6 @@
50975097
| ir.cpp:1051:39:1051:39 | SideEffect | ~m1051_20 |
50985098
| ir.cpp:1051:46:1051:46 | Address | &:r1051_9 |
50995099
| ir.cpp:1051:51:1051:70 | Address | &:r1051_11 |
5100-
| ir.cpp:1051:58:1051:58 | Address | &:r1051_16 |
5101-
| ir.cpp:1051:58:1051:58 | Arg(this) | this:r1051_16 |
5102-
| ir.cpp:1051:58:1051:58 | SideEffect | ~m1051_20 |
51035100
| ir.cpp:1051:58:1051:69 | Address | &:r1051_22 |
51045101
| ir.cpp:1051:58:1051:69 | Address | &:r1051_24 |
51055102
| ir.cpp:1051:58:1051:69 | Address | &:r1051_26 |
@@ -5120,6 +5117,9 @@
51205117
| ir.cpp:1051:60:1051:64 | SideEffect | ~m1051_4 |
51215118
| ir.cpp:1051:60:1051:64 | Unary | r1051_13 |
51225119
| ir.cpp:1051:60:1051:64 | Unary | r1051_15 |
5120+
| ir.cpp:1051:67:1051:67 | Address | &:r1051_16 |
5121+
| ir.cpp:1051:67:1051:67 | Arg(this) | this:r1051_16 |
5122+
| ir.cpp:1051:67:1051:67 | SideEffect | ~m1051_20 |
51235123
| ir.cpp:1051:73:1051:73 | ChiPartial | partial:m1051_9 |
51245124
| ir.cpp:1051:73:1051:73 | ChiTotal | total:m1051_3 |
51255125
| ir.cpp:1051:73:1051:73 | StoreValue | r1051_8 |
@@ -5184,9 +5184,6 @@
51845184
| ir.cpp:1054:49:1054:49 | SideEffect | ~m1054_20 |
51855185
| ir.cpp:1054:56:1054:56 | Address | &:r1054_9 |
51865186
| ir.cpp:1054:61:1054:88 | Address | &:r1054_11 |
5187-
| ir.cpp:1054:68:1054:68 | Address | &:r1054_16 |
5188-
| ir.cpp:1054:68:1054:68 | Arg(this) | this:r1054_16 |
5189-
| ir.cpp:1054:68:1054:68 | SideEffect | ~m1054_20 |
51905187
| ir.cpp:1054:68:1054:87 | Address | &:r1054_37 |
51915188
| ir.cpp:1054:68:1054:87 | Load | ~m1054_20 |
51925189
| ir.cpp:1054:68:1054:87 | StoreValue | r1054_38 |
@@ -5201,6 +5198,9 @@
52015198
| ir.cpp:1054:70:1054:74 | SideEffect | ~m1054_4 |
52025199
| ir.cpp:1054:70:1054:74 | Unary | r1054_13 |
52035200
| ir.cpp:1054:70:1054:74 | Unary | r1054_15 |
5201+
| ir.cpp:1054:77:1054:77 | Address | &:r1054_16 |
5202+
| ir.cpp:1054:77:1054:77 | Arg(this) | this:r1054_16 |
5203+
| ir.cpp:1054:77:1054:77 | SideEffect | ~m1054_20 |
52045204
| ir.cpp:1054:78:1054:82 | Address | &:r1054_22 |
52055205
| ir.cpp:1054:78:1054:82 | Address | &:r1054_24 |
52065206
| ir.cpp:1054:78:1054:82 | Left | r1054_25 |

cpp/ql/test/library-tests/lambdas/captures/elements.expected

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,10 +156,10 @@
156156
| captures.cpp:23:12:23:16 | x |
157157
| captures.cpp:23:12:23:16 | y |
158158
| captures.cpp:23:12:23:20 | ... + ... |
159-
| captures.cpp:23:16:23:16 | (reference dereference) |
160159
| captures.cpp:23:16:23:16 | definition of y |
161160
| captures.cpp:23:16:23:16 | y |
162161
| captures.cpp:23:16:23:16 | y |
162+
| captures.cpp:23:18:23:18 | (reference dereference) |
163163
| captures.cpp:23:20:23:20 | z |
164164
| captures.cpp:26:3:26:24 | return ... |
165165
| captures.cpp:26:10:26:17 | (const lambda [] type at line 22, col. 19)... |

cpp/ql/test/library-tests/sideEffects/stmts/stmts.expected

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
| stmts.c:3:5:3:10 | declaration | true | true |
33
| stmts.c:4:5:4:18 | declaration | true | true |
44
| stmts.c:5:5:5:16 | declaration | false | true |
5+
| stmts.c:7:5:7:5 | { ... } | true | true |
56
| stmts.c:7:5:14:5 | switch (...) ... | true | true |
67
| stmts.c:7:15:14:5 | { ... } | true | true |
78
| stmts.c:8:9:8:15 | case ...: | true | true |

0 commit comments

Comments
 (0)