aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Kelley <superjoe30@gmail.com>2017-08-05 16:52:19 -0400
committerAndrew Kelley <superjoe30@gmail.com>2017-08-05 16:54:50 -0400
commitad9f48b74bb5d43b767bdf401002d8bfd57c8813 (patch)
treeaf4f30fc3ca17ccdcd2824ce23ec859dcc24e191
parent27e4893ee59b965e4c030c43782798db258438ba (diff)
downloadzig-ad9f48b74bb5d43b767bdf401002d8bfd57c8813.tar.gz
zig-ad9f48b74bb5d43b767bdf401002d8bfd57c8813.zip
fix initializing undefined and crash when casting to invalid type
closes #408
-rw-r--r--src/analyze.cpp3
-rw-r--r--src/ir.cpp4
-rw-r--r--test/compile_errors.zig13
3 files changed, 20 insertions, 0 deletions
diff --git a/src/analyze.cpp b/src/analyze.cpp
index 574fa74879..6ad5af782b 100644
--- a/src/analyze.cpp
+++ b/src/analyze.cpp
@@ -3704,6 +3704,9 @@ void init_const_undefined(CodeGen *g, ConstExprValue *const_val) {
const_val->data.x_array.special = ConstArraySpecialUndef;
} else if (wanted_type->id == TypeTableEntryIdStruct) {
ensure_complete_type(g, wanted_type);
+ if (type_is_invalid(wanted_type)) {
+ return;
+ }
const_val->special = ConstValSpecialStatic;
size_t field_count = wanted_type->data.structure.src_field_count;
diff --git a/src/ir.cpp b/src/ir.cpp
index 0f6b70316e..6525036d9e 100644
--- a/src/ir.cpp
+++ b/src/ir.cpp
@@ -6945,6 +6945,9 @@ IrInstruction *ir_eval_const_value(CodeGen *codegen, Scope *scope, AstNode *node
FnTableEntry *fn_entry, Buf *c_import_buf, AstNode *source_node, Buf *exec_name,
IrExecutable *parent_exec)
{
+ if (expected_type != nullptr && type_is_invalid(expected_type))
+ return codegen->invalid_instruction;
+
IrExecutable ir_executable = {0};
ir_executable.source_node = source_node;
ir_executable.parent_exec = parent_exec;
@@ -14112,6 +14115,7 @@ TypeTableEntry *ir_analyze(CodeGen *codegen, IrExecutable *old_exec, IrExecutabl
TypeTableEntry *expected_type, AstNode *expected_type_source_node)
{
assert(!old_exec->invalid);
+ assert(expected_type == nullptr || !type_is_invalid(expected_type));
IrAnalyze ir_analyze_data = {};
IrAnalyze *ira = &ir_analyze_data;
diff --git a/test/compile_errors.zig b/test/compile_errors.zig
index 03e29e3450..e4b42b12a4 100644
--- a/test/compile_errors.zig
+++ b/test/compile_errors.zig
@@ -1918,4 +1918,17 @@ pub fn addCases(cases: &tests.CompileErrorContext) {
".tmp_source.zig:1:13: error: aoeu",
".tmp_source.zig:3:19: note: referenced here",
".tmp_source.zig:7:12: note: referenced here");
+
+ cases.add("instantiating an undefined value for an invalid struct that contains itself",
+ \\const Foo = struct {
+ \\ x: Foo,
+ \\};
+ \\
+ \\var foo: Foo = undefined;
+ \\
+ \\export fn entry() -> usize {
+ \\ return @sizeOf(@typeOf(foo.x));
+ \\}
+ ,
+ ".tmp_source.zig:1:13: error: struct 'Foo' contains itself");
}