Skip to content

Commit fb0f9ec

Browse files
committed
CPP: Add missing rnage analysis ranges in test.
1 parent 3ac3169 commit fb0f9ec

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

cpp/ql/test/library-tests/ir/range-analysis/SimpleRangeAnalysis_tests.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1039,7 +1039,7 @@ void test_guard_after_use(int pos, int size, int offset) {
10391039
if (offset == 1) {
10401040
return;
10411041
}
1042-
range(pos + 1); // $ overflow=+ range="==InitializeParameter: pos+1"
1042+
range(pos + 1); // $ overflow=+ range="==InitializeParameter: pos+1" MISSING: range="<=InitializeParameter: size-1"
10431043
}
10441044

10451045
int cond();
@@ -1060,8 +1060,8 @@ void alloc_in_loop(int origLen) {
10601060
}
10611061
len = len * 2; // $ overflow=-
10621062
}
1063-
// We want that index <= len
1064-
range(index);
1063+
// We want that index < len
1064+
range(index); // $ MISSING: range="<=InitializeParameter: len-1"
10651065
index++;
10661066
}
10671067
}
@@ -1079,7 +1079,7 @@ void mask_at_start(int len) {
10791079
for (int index = leftOver; index < len; index+=64) {
10801080
range(index); // $ range="<=InitializeParameter: len-1"
10811081
// This should be in bounds
1082-
range(index + 16); // $ range="<=InitializeParameter: len+15" range="==Phi: index+16"
1082+
range(index + 16); // $ range="<=InitializeParameter: len+15" range="==Phi: index+16" MISSING: range="<=InitializeParameter: len-1"
10831083
}
10841084
}
10851085

@@ -1096,7 +1096,7 @@ void mod_at_start(int len) {
10961096
// Do something with leftOver
10971097
for (int index = leftOver; index < len; index+=64) {
10981098
range(index); // $ range="<=InitializeParameter: len-1"
1099-
// This should be in bounds
1100-
range(index + 16); // $ range="<=InitializeParameter: len+15" range="==Phi: index+16"
1099+
// This should be in bounds
1100+
range(index + 16); // $ range="<=InitializeParameter: len+15" range="==Phi: index+16" MISSING: range="<=InitializeParameter: len-49"
11011101
}
11021102
}

0 commit comments

Comments
 (0)