diff --git a/lib/std/zig/string_literal.zig b/lib/std/zig/string_literal.zig index cc6030ad15c5..9def0bdefc59 100644 --- a/lib/std/zig/string_literal.zig +++ b/lib/std/zig/string_literal.zig @@ -104,7 +104,6 @@ pub fn parse( return error.InvalidCharacter; }, }, - else => unreachable, } } unreachable; diff --git a/src-self-hosted/translate_c.zig b/src-self-hosted/translate_c.zig index 3b1a91b28c32..dad3b14a2921 100644 --- a/src-self-hosted/translate_c.zig +++ b/src-self-hosted/translate_c.zig @@ -2791,7 +2791,6 @@ fn transCharLiteral( "TODO: support character literal kind {}", .{kind}, ), - else => unreachable, }; if (suppress_as == .no_as) { return maybeSuppressResult(rp, scope, result_used, int_lit_node); diff --git a/src/all_types.hpp b/src/all_types.hpp index 88c7e96943f9..ef60a05fab64 100644 --- a/src/all_types.hpp +++ b/src/all_types.hpp @@ -4095,7 +4095,7 @@ struct IrInstSrcCheckSwitchProngs { IrInstSrc *target_value; IrInstSrcCheckSwitchProngsRange *ranges; size_t range_count; - bool have_else_prong; + AstNode* else_prong; bool have_underscore_prong; }; diff --git a/src/ir.cpp b/src/ir.cpp index 635af397c4e1..8395cf0a3ee0 100644 --- a/src/ir.cpp +++ b/src/ir.cpp @@ -4274,14 +4274,14 @@ static IrInstGen *ir_build_err_to_int_gen(IrAnalyze *ira, Scope *scope, AstNode static IrInstSrc *ir_build_check_switch_prongs(IrBuilderSrc *irb, Scope *scope, AstNode *source_node, IrInstSrc *target_value, IrInstSrcCheckSwitchProngsRange *ranges, size_t range_count, - bool have_else_prong, bool have_underscore_prong) + AstNode* else_prong, bool have_underscore_prong) { IrInstSrcCheckSwitchProngs *instruction = ir_build_instruction( irb, scope, source_node); instruction->target_value = target_value; instruction->ranges = ranges; instruction->range_count = range_count; - instruction->have_else_prong = have_else_prong; + instruction->else_prong = else_prong; instruction->have_underscore_prong = have_underscore_prong; ir_ref_instruction(target_value, irb->current_basic_block); @@ -9305,7 +9305,7 @@ static IrInstSrc *ir_gen_switch_expr(IrBuilderSrc *irb, Scope *scope, AstNode *n } IrInstSrc *switch_prongs_void = ir_build_check_switch_prongs(irb, scope, node, target_value, - check_ranges.items, check_ranges.length, else_prong != nullptr, underscore_prong != nullptr); + check_ranges.items, check_ranges.length, else_prong, underscore_prong != nullptr); IrInstSrc *br_instruction; if (cases.length == 0) { @@ -28685,7 +28685,7 @@ static IrInstGen *ir_analyze_instruction_check_switch_prongs(IrAnalyze *ira, buf_ptr(enum_field->name))); } } - } else if (!instruction->have_else_prong) { + } else if (instruction->else_prong == nullptr) { if (switch_type->data.enumeration.non_exhaustive) { ir_add_error(ira, &instruction->base.base, buf_sprintf("switch on non-exhaustive enum must include `else` or `_` prong")); @@ -28700,6 +28700,10 @@ static IrInstGen *ir_analyze_instruction_check_switch_prongs(IrAnalyze *ira, buf_ptr(enum_field->name))); } } + } else if(!switch_type->data.enumeration.non_exhaustive && switch_type->data.enumeration.src_field_count == instruction->range_count) { + ir_add_error_node(ira, instruction->else_prong, + buf_sprintf("unreachable else prong, all cases already handled")); + return ira->codegen->invalid_inst_gen; } } else if (switch_type->id == ZigTypeIdErrorSet) { if (!resolve_inferred_error_set(ira->codegen, switch_type, target_value->base.source_node)) { @@ -28746,7 +28750,7 @@ static IrInstGen *ir_analyze_instruction_check_switch_prongs(IrAnalyze *ira, } field_prev_uses[start_index] = start_value->base.source_node; } - if (!instruction->have_else_prong) { + if (instruction->else_prong == nullptr) { if (type_is_global_error_set(switch_type)) { ir_add_error(ira, &instruction->base.base, buf_sprintf("else prong required when switching on type 'anyerror'")); @@ -28808,16 +28812,20 @@ static IrInstGen *ir_analyze_instruction_check_switch_prongs(IrAnalyze *ira, return ira->codegen->invalid_inst_gen; } } - if (!instruction->have_else_prong) { + BigInt min_val; eval_min_max_value_int(ira->codegen, switch_type, &min_val, false); BigInt max_val; eval_min_max_value_int(ira->codegen, switch_type, &max_val, true); - if (!rangeset_spans(&rs, &min_val, &max_val)) { + bool handles_all_cases = rangeset_spans(&rs, &min_val, &max_val); + if (!handles_all_cases && instruction->else_prong == nullptr) { ir_add_error(ira, &instruction->base.base, buf_sprintf("switch must handle all possibilities")); return ira->codegen->invalid_inst_gen; + } else if(handles_all_cases && instruction->else_prong != nullptr) { + ir_add_error_node(ira, instruction->else_prong, + buf_sprintf("unreachable else prong, all cases already handled")); + return ira->codegen->invalid_inst_gen; } - } } else if (switch_type->id == ZigTypeIdBool) { int seenTrue = 0; int seenFalse = 0; @@ -28847,11 +28855,17 @@ static IrInstGen *ir_analyze_instruction_check_switch_prongs(IrAnalyze *ira, return ira->codegen->invalid_inst_gen; } } - if (((seenTrue < 1) || (seenFalse < 1)) && !instruction->have_else_prong) { + if (((seenTrue < 1) || (seenFalse < 1)) && instruction->else_prong == nullptr) { ir_add_error(ira, &instruction->base.base, buf_sprintf("switch must handle all possibilities")); return ira->codegen->invalid_inst_gen; } - } else if (!instruction->have_else_prong) { + + if(seenTrue == 1 && seenFalse == 1 && instruction->else_prong != nullptr) { + ir_add_error_node(ira, instruction->else_prong, + buf_sprintf("unreachable else prong, all cases already handled")); + return ira->codegen->invalid_inst_gen; + } + } else if (instruction->else_prong == nullptr) { ir_add_error(ira, &instruction->base.base, buf_sprintf("else prong required when switching on type '%s'", buf_ptr(&switch_type->name))); return ira->codegen->invalid_inst_gen; diff --git a/src/ir_print.cpp b/src/ir_print.cpp index 27bedff47f45..e9f029c9307a 100644 --- a/src/ir_print.cpp +++ b/src/ir_print.cpp @@ -2177,7 +2177,7 @@ static void ir_print_check_switch_prongs(IrPrintSrc *irp, IrInstSrcCheckSwitchPr fprintf(irp->f, "..."); ir_print_other_inst_src(irp, instruction->ranges[i].end); } - const char *have_else_str = instruction->have_else_prong ? "yes" : "no"; + const char *have_else_str = instruction->else_prong != nullptr ? "yes" : "no"; fprintf(irp->f, ")else:%s", have_else_str); } diff --git a/test/compile_errors.zig b/test/compile_errors.zig index 3f898cc33773..29ff994cd357 100644 --- a/test/compile_errors.zig +++ b/test/compile_errors.zig @@ -468,6 +468,102 @@ pub fn addCases(cases: *tests.CompileErrorContext) void { "tmp.zig:12:5: error: switch on non-exhaustive enum must include `else` or `_` prong", }); + cases.add("switch expression - unreachable else prong (bool)", + \\fn foo(x: bool) void { + \\ switch (x) { + \\ true => {}, + \\ false => {}, + \\ else => {}, + \\ } + \\} + \\export fn entry() usize { return @sizeOf(@TypeOf(foo)); } + , &[_][]const u8{ + "tmp.zig:5:9: error: unreachable else prong, all cases already handled", + }); + + cases.add("switch expression - unreachable else prong (u1)", + \\fn foo(x: u1) void { + \\ switch (x) { + \\ 0 => {}, + \\ 1 => {}, + \\ else => {}, + \\ } + \\} + \\export fn entry() usize { return @sizeOf(@TypeOf(foo)); } + , &[_][]const u8{ + "tmp.zig:5:9: error: unreachable else prong, all cases already handled", + }); + + cases.add("switch expression - unreachable else prong (u2)", + \\fn foo(x: u2) void { + \\ switch (x) { + \\ 0 => {}, + \\ 1 => {}, + \\ 2 => {}, + \\ 3 => {}, + \\ else => {}, + \\ } + \\} + \\export fn entry() usize { return @sizeOf(@TypeOf(foo)); } + , &[_][]const u8{ + "tmp.zig:7:9: error: unreachable else prong, all cases already handled", + }); + + cases.add("switch expression - unreachable else prong (range u8)", + \\fn foo(x: u8) void { + \\ switch (x) { + \\ 0 => {}, + \\ 1 => {}, + \\ 2 => {}, + \\ 3 => {}, + \\ 4...255 => {}, + \\ else => {}, + \\ } + \\} + \\export fn entry() usize { return @sizeOf(@TypeOf(foo)); } + , &[_][]const u8{ + "tmp.zig:8:9: error: unreachable else prong, all cases already handled", + }); + + cases.add("switch expression - unreachable else prong (range i8)", + \\fn foo(x: i8) void { + \\ switch (x) { + \\ -128...0 => {}, + \\ 1 => {}, + \\ 2 => {}, + \\ 3 => {}, + \\ 4...127 => {}, + \\ else => {}, + \\ } + \\} + \\export fn entry() usize { return @sizeOf(@TypeOf(foo)); } + , &[_][]const u8{ + "tmp.zig:8:9: error: unreachable else prong, all cases already handled", + }); + + cases.add("switch expression - unreachable else prong (enum)", + \\const TestEnum = enum{ T1, T2 }; + \\ + \\fn err(x: u8) TestEnum { + \\ switch (x) { + \\ 0 => return TestEnum.T1, + \\ else => return TestEnum.T2, + \\ } + \\} + \\ + \\fn foo(x: u8) void { + \\ switch (err(x)) { + \\ TestEnum.T1 => {}, + \\ TestEnum.T2 => {}, + \\ else => {}, + \\ } + \\} + \\ + \\export fn entry() usize { return @sizeOf(@TypeOf(foo)); } + , &[_][]const u8{ + "tmp.zig:14:9: error: unreachable else prong, all cases already handled", + }); + cases.addTest("@export with empty name string", \\pub export fn entry() void { } \\comptime { diff --git a/test/stage1/behavior/bugs/1111.zig b/test/stage1/behavior/bugs/1111.zig index f62107f9a331..607bc33666dc 100644 --- a/test/stage1/behavior/bugs/1111.zig +++ b/test/stage1/behavior/bugs/1111.zig @@ -7,6 +7,5 @@ test "issue 1111 fixed" { switch (v) { Foo.Bar => return, - else => return, } }