1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
|
//: Some simple sanity checks for types, and also attempts to guess them where
//: they aren't provided.
//:
//: You still have to provide the full type the first time you mention a
//: variable in a recipe. You have to explicitly name :offset and :variant
//: every single time. You can't use the same name with multiple types in a
//: single recipe.
:(scenario transform_fails_on_reusing_name_with_different_type)
% Hide_errors = true;
def main [
x:num <- copy 1
x:bool <- copy 1
]
+error: main: 'x' used with multiple types
//: we need surrounding-space info for type-checking variables in other spaces
:(after "Transform.push_back(collect_surrounding_spaces)")
Transform.push_back(check_or_set_types_by_name); // idempotent
// Keep the name->type mapping for all recipes around for the entire
// transformation phase.
:(before "End Globals")
map<recipe_ordinal, set<reagent, name_lt> > Types_by_space; // internal to transform; no need to snapshot
:(before "End Reset")
Types_by_space.clear();
:(before "End transform_all")
Types_by_space.clear();
:(before "End Types")
struct name_lt {
bool operator()(const reagent& a, const reagent& b) const { return a.name < b.name; }
};
:(code)
void check_or_set_types_by_name(const recipe_ordinal r) {
recipe& caller = get(Recipe, r);
trace(9991, "transform") << "--- deduce types for recipe " << caller.name << end();
for (int i = 0; i < SIZE(caller.steps); ++i) {
instruction& inst = caller.steps.at(i);
for (int in = 0; in < SIZE(inst.ingredients); ++in)
check_or_set_type(inst.ingredients.at(in), caller);
for (int out = 0; out < SIZE(inst.products); ++out)
check_or_set_type(inst.products.at(out), caller);
}
}
void check_or_set_type(reagent& curr, const recipe& caller) {
if (is_literal(curr)) return;
if (is_integer(curr.name)) return; // no type-checking for raw locations
set<reagent, name_lt>& known_types = Types_by_space[owning_recipe(curr, caller.ordinal)];
deduce_missing_type(known_types, curr, caller);
check_type(known_types, curr, caller);
}
void deduce_missing_type(set<reagent, name_lt>& known_types, reagent& x, const recipe& caller) {
// Deduce Missing Type(x, caller)
if (x.type) return;
if (is_jump_target(x.name)) {
x.type = new type_tree("label");
return;
}
if (known_types.find(x) == known_types.end()) return;
const reagent& exemplar = *known_types.find(x);
x.type = new type_tree(*exemplar.type);
trace(9992, "transform") << x.name << " <= " << names_to_string(x.type) << end();
// spaces are special; their type includes their /names property
if (is_mu_space(x) && !has_property(x, "names")) {
if (!has_property(exemplar, "names")) {
raise << maybe(caller.name) << "missing /names property for space variable '" << exemplar.name << "'\n" << end();
return;
}
x.properties.push_back(pair<string, string_tree*>("names", new string_tree(*property(exemplar, "names"))));
}
}
void check_type(set<reagent, name_lt>& known_types, const reagent& x, const recipe& caller) {
if (is_literal(x)) return;
if (!x.type) return; // might get filled in by other logic later
if (is_jump_target(x.name)) {
if (!x.type->atom || x.type->name != "label")
raise << maybe(caller.name) << "non-label '" << x.name << "' must begin with a letter\n" << end();
return;
}
if (known_types.find(x) == known_types.end()) {
trace(9992, "transform") << x.name << " => " << names_to_string(x.type) << end();
known_types.insert(x);
}
if (!types_strictly_match(known_types.find(x)->type, x.type)) {
raise << maybe(caller.name) << "'" << x.name << "' used with multiple types\n" << end();
raise << " " << to_string(known_types.find(x)->type) << " vs " << to_string(x.type) << '\n' << end();
return;
}
if (is_mu_array(x)) {
if (!x.type->right) {
raise << maybe(caller.name) << "'" << x.name << ": can't be just an array. What is it an array of?\n" << end();
return;
}
if (!x.type->right->right) {
raise << caller.name << " can't determine the size of array variable '" << x.name << "'. Either allocate it separately and make the type of '" << x.name << "' an address, or specify the length of the array in the type of '" << x.name << "'.\n" << end();
return;
}
}
}
recipe_ordinal owning_recipe(const reagent& x, recipe_ordinal r) {
for (int s = space_index(x); s > 0; --s) {
if (!contains_key(Surrounding_space, r)) break; // error raised elsewhere
r = Surrounding_space[r];
}
return r;
}
:(scenario transform_fills_in_missing_types)
def main [
x:num <- copy 11
y:num <- add x, 1
]
# x is in location 2, y in location 3
+mem: storing 12 in location 3
:(scenario transform_fills_in_missing_types_in_product)
def main [
x:num <- copy 11
x <- copy 12
]
# x is in location 2
+mem: storing 12 in location 2
:(scenario transform_fills_in_missing_types_in_product_and_ingredient)
def main [
x:num <- copy 11
x <- add x, 1
]
# x is in location 2
+mem: storing 12 in location 2
:(scenario transform_fills_in_missing_label_type)
def main [
jump +target
1:num <- copy 0
+target
]
-mem: storing 0 in location 1
:(scenario transform_fails_on_missing_types_in_first_mention)
% Hide_errors = true;
def main [
x <- copy 1
x:num <- copy 2
]
+error: main: missing type for 'x' in 'x <- copy 1'
:(scenario transform_fails_on_wrong_type_for_label)
% Hide_errors = true;
def main [
+foo:num <- copy 34
]
+error: main: non-label '+foo' must begin with a letter
:(scenario typo_in_address_type_fails)
% Hide_errors = true;
def main [
y:&:charcter <- new character:type
*y <- copy 67
]
+error: main: unknown type charcter in 'y:&:charcter <- new character:type'
:(scenario array_type_without_size_fails)
% Hide_errors = true;
def main [
x:@:num <- merge 2, 12, 13
]
+error: main can't determine the size of array variable 'x'. Either allocate it separately and make the type of 'x' an address, or specify the length of the array in the type of 'x'.
:(scenarios transform)
:(scenario transform_checks_types_of_identical_reagents_in_multiple_spaces)
def foo [ # dummy
]
def main [
local-scope
0:space/names:foo <- copy null # specify surrounding space
x:bool <- copy true
x:num/space:1 <- copy 34
x/space:1 <- copy 35
]
$error: 0
:(scenario transform_handles_empty_reagents)
% Hide_errors = true;
def main [
add *
]
+error: illegal name '*'
# no crash
:(scenario transform_checks_types_in_surrounding_spaces)
% Hide_errors = true;
# 'x' is a bool in foo's space
def foo [
local-scope
x:bool <- copy false
return default-space/names:foo
]
# try to read 'x' as a num in foo's space
def main [
local-scope
0:space/names:foo <- foo
x:num/space:1 <- copy 34
]
error: foo: 'x' used with multiple types
|