author | Mario de Sousa <msousa@fe.up.pt> |
Tue, 12 Jun 2012 17:15:24 +0100 | |
changeset 586 | b602f0459f17 |
parent 585 | be7330d9b65c |
child 592 | 99a284cec1f2 |
permissions | -rw-r--r-- |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
1 |
/* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
2 |
* matiec - a compiler for the programming languages defined in IEC 61131-3 |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
3 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
4 |
* Copyright (C) 2009-2012 Mario de Sousa (msousa@fe.up.pt) |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
5 |
* Copyright (C) 2012 Manuele Conti (conti.ma@alice.it) |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
6 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
7 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
8 |
* This program is free software: you can redistribute it and/or modify |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
9 |
* it under the terms of the GNU General Public License as published by |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
10 |
* the Free Software Foundation, either version 3 of the License, or |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
11 |
* (at your option) any later version. |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
12 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
13 |
* This program is distributed in the hope that it will be useful, |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
14 |
* but WITHOUT ANY WARRANTY; without even the implied warranty of |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
15 |
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
16 |
* GNU General Public License for more details. |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
17 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
18 |
* You should have received a copy of the GNU General Public License |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
19 |
* along with this program. If not, see <http://www.gnu.org/licenses/>. |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
20 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
21 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
22 |
* This code is made available on the understanding that it will not be |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
23 |
* used in safety-critical situations without a full and competent review. |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
24 |
*/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
25 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
26 |
/* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
27 |
* An IEC 61131-3 compiler. |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
28 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
29 |
* Based on the |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
30 |
* FINAL DRAFT - IEC 61131-3, 2nd Ed. (2001-12-10) |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
31 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
32 |
*/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
33 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
34 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
35 |
/* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
36 |
* TODO: |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
37 |
* - Check subscript values fall within allowed range. |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
38 |
* For the checking of subscript values to work correctly, we would need to have constant folding working too: |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
39 |
* array_var[8 + 99] can not be checked without constant folding. |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
40 |
* However, even without constant folding range check may be usefull, |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
41 |
* and later changing it to use the values coming out of constant folding should not be very difficult. |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
42 |
* |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
43 |
*/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
44 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
45 |
|
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
46 |
#include "array_range_check.hh" |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
47 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
48 |
#define FIRST_(symbol1, symbol2) (((symbol1)->first_order < (symbol2)->first_order) ? (symbol1) : (symbol2)) |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
49 |
#define LAST_(symbol1, symbol2) (((symbol1)->last_order > (symbol2)->last_order) ? (symbol1) : (symbol2)) |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
50 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
51 |
#define STAGE3_ERROR(error_level, symbol1, symbol2, ...) { \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
52 |
if (current_display_error_level >= error_level) { \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
53 |
fprintf(stderr, "%s:%d-%d..%d-%d: error: ", \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
54 |
FIRST_(symbol1,symbol2)->first_file, FIRST_(symbol1,symbol2)->first_line, FIRST_(symbol1,symbol2)->first_column,\ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
55 |
LAST_(symbol1,symbol2) ->last_line, LAST_(symbol1,symbol2) ->last_column);\ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
56 |
fprintf(stderr, __VA_ARGS__); \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
57 |
fprintf(stderr, "\n"); \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
58 |
error_count++; \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
59 |
} \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
60 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
61 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
62 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
63 |
#define STAGE3_WARNING(symbol1, symbol2, ...) { \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
64 |
fprintf(stderr, "%s:%d-%d..%d-%d: warning: ", \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
65 |
FIRST_(symbol1,symbol2)->first_file, FIRST_(symbol1,symbol2)->first_line, FIRST_(symbol1,symbol2)->first_column,\ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
66 |
LAST_(symbol1,symbol2) ->last_line, LAST_(symbol1,symbol2) ->last_column);\ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
67 |
fprintf(stderr, __VA_ARGS__); \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
68 |
fprintf(stderr, "\n"); \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
69 |
warning_found = true; \ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
70 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
71 |
|
581
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
72 |
#define GET_CVALUE(dtype, symbol) ((symbol)->const_value_##dtype->value) |
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
73 |
#define VALID_CVALUE(dtype, symbol) ((NULL != (symbol)->const_value_##dtype) && (symbol_c::cs_const_value == (symbol)->const_value_##dtype->status)) |
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
74 |
|
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
75 |
|
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
76 |
|
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
77 |
array_range_check_c::array_range_check_c(symbol_c *ignore) { |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
78 |
error_count = 0; |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
79 |
current_display_error_level = 0; |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
80 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
81 |
|
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
82 |
array_range_check_c::~array_range_check_c(void) { |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
83 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
84 |
|
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
85 |
int array_range_check_c::get_error_count() { |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
86 |
return error_count; |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
87 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
88 |
|
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
89 |
void array_range_check_c::check_dimension_count(array_variable_c *symbol) { |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
90 |
int dimension_count; |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
91 |
symbol_c *var_decl; |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
92 |
|
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
93 |
var_decl = search_varfb_instance_type->get_basetype_decl(symbol->subscripted_variable); |
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
94 |
array_dimension_iterator_c array_dimension_iterator(var_decl); |
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
95 |
for (dimension_count = 0; NULL != array_dimension_iterator.next(); dimension_count++); |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
96 |
if (dimension_count != ((list_c *)symbol->subscript_list)->n) |
583
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
97 |
STAGE3_ERROR(0, symbol, symbol, "Number of subscripts/indexes does not match the number of subscripts/indexes in the array's declaration (array has %d indexes)", dimension_count); |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
98 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
99 |
|
581
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
100 |
void array_range_check_c::check_bounds(array_variable_c *symbol) { |
583
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
101 |
list_c *l; /* the subscript_list */ |
581
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
102 |
symbol_c *var_decl; |
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
103 |
|
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
104 |
l = (list_c *)symbol->subscript_list; |
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
105 |
var_decl = search_varfb_instance_type->get_basetype_decl(symbol->subscripted_variable); |
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
106 |
array_dimension_iterator_c array_dimension_iterator(var_decl); |
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
107 |
for (int i = 0; i < l->n; i++) { |
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
108 |
subrange_c *dimension = array_dimension_iterator.next(); |
585
be7330d9b65c
Remove potential access to NULL pointer.
Mario de Sousa <msousa@fe.up.pt>
parents:
583
diff
changeset
|
109 |
/* mismatch between number of indexes/subscripts. This error will be caught in check_dimension_count() so we ignore it. */ |
be7330d9b65c
Remove potential access to NULL pointer.
Mario de Sousa <msousa@fe.up.pt>
parents:
583
diff
changeset
|
110 |
if (NULL == dimension) |
be7330d9b65c
Remove potential access to NULL pointer.
Mario de Sousa <msousa@fe.up.pt>
parents:
583
diff
changeset
|
111 |
return; |
583
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
112 |
|
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
113 |
if ( VALID_CVALUE( int64, l->elements[i]) && VALID_CVALUE( int64, dimension->lower_limit)) |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
114 |
if ( GET_CVALUE( int64, l->elements[i]) < GET_CVALUE( int64, dimension->lower_limit)) |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
115 |
{STAGE3_ERROR(0, symbol, symbol, "Array access out of bounds."); continue;} |
581
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
116 |
|
583
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
117 |
if ( VALID_CVALUE( int64, l->elements[i]) && VALID_CVALUE( int64, dimension->upper_limit)) |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
118 |
if ( GET_CVALUE( int64, l->elements[i]) > GET_CVALUE( int64, dimension->upper_limit)) |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
119 |
{STAGE3_ERROR(0, symbol, symbol, "Array access out of bounds."); continue;} |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
120 |
|
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
121 |
if ( VALID_CVALUE(uint64, l->elements[i]) && VALID_CVALUE(uint64, dimension->lower_limit)) |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
122 |
if ( GET_CVALUE(uint64, l->elements[i]) < GET_CVALUE(uint64, dimension->lower_limit)) |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
123 |
{STAGE3_ERROR(0, symbol, symbol, "Array access out of bounds."); continue;} |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
124 |
|
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
125 |
if ( VALID_CVALUE(uint64, l->elements[i]) && VALID_CVALUE(uint64, dimension->upper_limit)) |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
126 |
if ( GET_CVALUE(uint64, l->elements[i]) > GET_CVALUE(uint64, dimension->upper_limit)) |
e1df3781be84
Fix array index bound check to consider uint64 indexes/limits.
Mario de Sousa <msousa@fe.up.pt>
parents:
581
diff
changeset
|
127 |
{STAGE3_ERROR(0, symbol, symbol, "Array access out of bounds."); continue;} |
581
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
128 |
} |
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
129 |
} |
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
130 |
|
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
131 |
/*********************/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
132 |
/* B 1.4 - Variables */ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
133 |
/*********************/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
134 |
/*************************************/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
135 |
/* B 1.4.2 - Multi-element variables */ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
136 |
/*************************************/ |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
137 |
void *array_range_check_c::visit(array_variable_c *symbol) { |
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
138 |
check_dimension_count(symbol); |
581
1e158dc9f9c1
Add check array out of bound.
Manuele Conti <conti.ma@alice.it>
parents:
560
diff
changeset
|
139 |
check_bounds(symbol); |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
140 |
return NULL; |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
141 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
142 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
143 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
144 |
/**************************************/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
145 |
/* B 1.5 - Program organisation units */ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
146 |
/**************************************/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
147 |
/***********************/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
148 |
/* B 1.5.1 - Functions */ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
149 |
/***********************/ |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
150 |
void *array_range_check_c::visit(function_declaration_c *symbol) { |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
151 |
search_varfb_instance_type = new search_varfb_instance_type_c(symbol); |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
152 |
// search_var_instance_decl = new search_var_instance_decl_c(symbol); |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
153 |
symbol->function_body->accept(*this); |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
154 |
delete search_varfb_instance_type; |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
155 |
// delete search_var_instance_decl; |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
156 |
search_varfb_instance_type = NULL; |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
157 |
// search_var_instance_decl = NULL; |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
158 |
return NULL; |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
159 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
160 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
161 |
/*****************************/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
162 |
/* B 1.5.2 - Function blocks */ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
163 |
/*****************************/ |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
164 |
void *array_range_check_c::visit(function_block_declaration_c *symbol) { |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
165 |
search_varfb_instance_type = new search_varfb_instance_type_c(symbol); |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
166 |
// search_var_instance_decl = new search_var_instance_decl_c(symbol); |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
167 |
symbol->fblock_body->accept(*this); |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
168 |
delete search_varfb_instance_type; |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
169 |
// delete search_var_instance_decl; |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
170 |
search_varfb_instance_type = NULL; |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
171 |
// search_var_instance_decl = NULL; |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
172 |
return NULL; |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
173 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
174 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
175 |
/**********************/ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
176 |
/* B 1.5.3 - Programs */ |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
177 |
/**********************/ |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
178 |
void *array_range_check_c::visit(program_declaration_c *symbol) { |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
179 |
search_varfb_instance_type = new search_varfb_instance_type_c(symbol); |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
180 |
// search_var_instance_decl = new search_var_instance_decl_c(symbol); |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
181 |
symbol->function_block_body->accept(*this); |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
182 |
delete search_varfb_instance_type; |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
183 |
// delete search_var_instance_decl; |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
184 |
search_varfb_instance_type = NULL; |
560
13b5b7faa3d7
Renamed class, remove unused code, delete allocated objects.
mjsousa <msousa@fe.up.pt>
parents:
559
diff
changeset
|
185 |
// search_var_instance_decl = NULL; |
559
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
186 |
return NULL; |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
187 |
} |
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
188 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
189 |
|
a3b8925e640c
Start sematinc range check class.
Manuele Conti <conti.ma@alice.it>
parents:
diff
changeset
|
190 |