authorgravatar for andrew@ziglang.orgAndrew Kelley <andrew@ziglang.org> 2018-08-04 15:19:03-04:00
committergravatar for andrew@ziglang.orgAndrew Kelley <andrew@ziglang.org> 2018-08-04 15:19:03-04:00
logb48948d6e805cdee7f33243098af9a862c152df1
tree8b1b4b37670f7444e8fe12748a8db99ac7498cce
parentf804310d9f953c9d78a4271ba8d75133341840e6
parent9bd8b01650f9cf21e601117951711b21aa5fd216

Merge branch 'master' into llvm7


32 files changed, 366 insertions(+), 176 deletions(-)

.gitignore+11
......@@ -1,3 +1,14 @@
1# This file is for zig-specific build artifacts.
2# If you have OS-specific or editor-specific files to ignore,
3# such as *.swp or .DS_Store, put those in your global
4# ~/.gitignore and put this in your ~/.gitconfig:
5#
6# [core]
7# excludesfile = ~/.gitignore
8#
9# Cheers!
10# -andrewrk
11
112zig-cache/
213build/
314build-*/
doc/langref.html.in+18-8
......@@ -4690,9 +4690,9 @@ test "coroutine suspend with block" {
46904690var a_promise: promise = undefined;
46914691var result = false;
46924692async fn testSuspendBlock() void {
4693 suspend |p| {
4694 comptime assert(@typeOf(p) == promise->void);
4695 a_promise = p;
4693 suspend {
4694 comptime assert(@typeOf(@handle()) == promise->void);
4695 a_promise = @handle();
46964696 }
46974697 result = true;
46984698}
......@@ -4733,8 +4733,8 @@ test "resume from suspend" {
47334733 std.debug.assert(my_result == 2);
47344734}
47354735async fn testResumeFromSuspend(my_result: *i32) void {
4736 suspend |p| {
4737 resume p;
4736 suspend {
4737 resume @handle();
47384738 }
47394739 my_result.* += 1;
47404740 suspend;
......@@ -4791,9 +4791,9 @@ async fn amain() void {
47914791}
47924792async fn another() i32 {
47934793 seq('c');
4794 suspend |p| {
4794 suspend {
47954795 seq('d');
4796 a_promise = p;
4796 a_promise = @handle();
47974797 }
47984798 seq('g');
47994799 return 1234;
......@@ -5383,6 +5383,16 @@ test "main" {
53835383 This function is only valid within function scope.
53845384 </p>
53855385 {#header_close#}
5386 {#header_open|@handle#}
5387 <pre><code class="zig">@handle()</code></pre>
5388 <p>
5389 This function returns a <code>promise->T</code> type, where <code>T</code>
5390 is the return type of the async function in scope.
5391 </p>
5392 <p>
5393 This function is only valid within an async function scope.
5394 </p>
5395 {#header_close#}
53865396 {#header_open|@import#}
53875397 <pre><code class="zig">@import(comptime path: []u8) (namespace)</code></pre>
53885398 <p>
......@@ -7388,7 +7398,7 @@ Defer(body) = ("defer" | "deferror") body
73887398
73897399IfExpression(body) = "if" "(" Expression ")" body option("else" BlockExpression(body))
73907400
7391SuspendExpression(body) = "suspend" option(("|" Symbol "|" body))
7401SuspendExpression(body) = "suspend" option( body )
73927402
73937403IfErrorExpression(body) = "if" "(" Expression ")" option("|" option("*") Symbol "|") body "else" "|" Symbol "|" BlockExpression(body)
73947404
src/all_types.hpp+7-1
......@@ -899,7 +899,6 @@ struct AstNodeAwaitExpr {
899899
900900struct AstNodeSuspend {
901901 AstNode *block;
902 AstNode *promise_symbol;
903902};
904903
905904struct AstNodePromiseType {
......@@ -1358,6 +1357,7 @@ enum BuiltinFnId {
13581357 BuiltinFnIdBreakpoint,
13591358 BuiltinFnIdReturnAddress,
13601359 BuiltinFnIdFrameAddress,
1360 BuiltinFnIdHandle,
13611361 BuiltinFnIdEmbedFile,
13621362 BuiltinFnIdCmpxchgWeak,
13631363 BuiltinFnIdCmpxchgStrong,
......@@ -1714,6 +1714,7 @@ struct CodeGen {
17141714 LLVMValueRef coro_save_fn_val;
17151715 LLVMValueRef coro_promise_fn_val;
17161716 LLVMValueRef coro_alloc_helper_fn_val;
1717 LLVMValueRef coro_frame_fn_val;
17171718 LLVMValueRef merge_err_ret_traces_fn_val;
17181719 LLVMValueRef add_error_return_trace_addr_fn_val;
17191720 LLVMValueRef stacksave_fn_val;
......@@ -2074,6 +2075,7 @@ enum IrInstructionId {
20742075 IrInstructionIdBreakpoint,
20752076 IrInstructionIdReturnAddress,
20762077 IrInstructionIdFrameAddress,
2078 IrInstructionIdHandle,
20772079 IrInstructionIdAlignOf,
20782080 IrInstructionIdOverflowOp,
20792081 IrInstructionIdTestErr,
......@@ -2791,6 +2793,10 @@ struct IrInstructionFrameAddress {
27912793 IrInstruction base;
27922794};
27932795
2796struct IrInstructionHandle {
2797 IrInstruction base;
2798};
2799
27942800enum IrOverflowOp {
27952801 IrOverflowOpAdd,
27962802 IrOverflowOpSub,
src/ast_render.cpp-3
......@@ -1112,9 +1112,6 @@ static void render_node_extra(AstRender *ar, AstNode *node, bool grouped) {
11121112 {
11131113 fprintf(ar->f, "suspend");
11141114 if (node->data.suspend.block != nullptr) {
1115 fprintf(ar->f, " |");
1116 render_node_grouped(ar, node->data.suspend.promise_symbol);
1117 fprintf(ar->f, "| ");
11181115 render_node_grouped(ar, node->data.suspend.block);
11191116 }
11201117 break;
src/codegen.cpp+24
......@@ -4057,6 +4057,26 @@ static LLVMValueRef ir_render_frame_address(CodeGen *g, IrExecutable *executable
40574057 return LLVMBuildCall(g->builder, get_frame_address_fn_val(g), &zero, 1, "");
40584058}
40594059
4060static LLVMValueRef get_handle_fn_val(CodeGen *g) {
4061 if (g->coro_frame_fn_val)
4062 return g->coro_frame_fn_val;
4063
4064 LLVMTypeRef fn_type = LLVMFunctionType( LLVMPointerType(LLVMInt8Type(), 0)
4065 , nullptr, 0, false);
4066 Buf *name = buf_sprintf("llvm.coro.frame");
4067 g->coro_frame_fn_val = LLVMAddFunction(g->module, buf_ptr(name), fn_type);
4068 assert(LLVMGetIntrinsicID(g->coro_frame_fn_val));
4069
4070 return g->coro_frame_fn_val;
4071}
4072
4073static LLVMValueRef ir_render_handle(CodeGen *g, IrExecutable *executable,
4074 IrInstructionHandle *instruction)
4075{
4076 LLVMValueRef zero = LLVMConstNull(g->builtin_types.entry_promise->type_ref);
4077 return LLVMBuildCall(g->builder, get_handle_fn_val(g), &zero, 0, "");
4078}
4079
40604080static LLVMValueRef render_shl_with_overflow(CodeGen *g, IrInstructionOverflowOp *instruction) {
40614081 TypeTableEntry *int_type = instruction->result_ptr_type;
40624082 assert(int_type->id == TypeTableEntryIdInt);
......@@ -4821,6 +4841,8 @@ static LLVMValueRef ir_render_instruction(CodeGen *g, IrExecutable *executable,
48214841 return ir_render_return_address(g, executable, (IrInstructionReturnAddress *)instruction);
48224842 case IrInstructionIdFrameAddress:
48234843 return ir_render_frame_address(g, executable, (IrInstructionFrameAddress *)instruction);
4844 case IrInstructionIdHandle:
4845 return ir_render_handle(g, executable, (IrInstructionHandle *)instruction);
48244846 case IrInstructionIdOverflowOp:
48254847 return ir_render_overflow_op(g, executable, (IrInstructionOverflowOp *)instruction);
48264848 case IrInstructionIdTestErr:
......@@ -5916,6 +5938,7 @@ static void do_code_gen(CodeGen *g) {
59165938 ir_render(g, fn_table_entry);
59175939
59185940 }
5941
59195942 assert(!g->errors.length);
59205943
59215944 if (buf_len(&g->global_asm) != 0) {
......@@ -6255,6 +6278,7 @@ static void define_builtin_fns(CodeGen *g) {
62556278 create_builtin_fn(g, BuiltinFnIdBreakpoint, "breakpoint", 0);
62566279 create_builtin_fn(g, BuiltinFnIdReturnAddress, "returnAddress", 0);
62576280 create_builtin_fn(g, BuiltinFnIdFrameAddress, "frameAddress", 0);
6281 create_builtin_fn(g, BuiltinFnIdHandle, "handle", 0);
62586282 create_builtin_fn(g, BuiltinFnIdMemcpy, "memcpy", 3);
62596283 create_builtin_fn(g, BuiltinFnIdMemset, "memset", 3);
62606284 create_builtin_fn(g, BuiltinFnIdSizeof, "sizeOf", 1);
src/ir.cpp+64-14
......@@ -580,6 +580,10 @@ static constexpr IrInstructionId ir_instruction_id(IrInstructionFrameAddress *)
580580 return IrInstructionIdFrameAddress;
581581}
582582
583static constexpr IrInstructionId ir_instruction_id(IrInstructionHandle *) {
584 return IrInstructionIdHandle;
585}
586
583587static constexpr IrInstructionId ir_instruction_id(IrInstructionAlignOf *) {
584588 return IrInstructionIdAlignOf;
585589}
......@@ -2240,6 +2244,17 @@ static IrInstruction *ir_build_frame_address_from(IrBuilder *irb, IrInstruction
22402244 return new_instruction;
22412245}
22422246
2247static IrInstruction *ir_build_handle(IrBuilder *irb, Scope *scope, AstNode *source_node) {
2248 IrInstructionHandle *instruction = ir_build_instruction<IrInstructionHandle>(irb, scope, source_node);
2249 return &instruction->base;
2250}
2251
2252static IrInstruction *ir_build_handle_from(IrBuilder *irb, IrInstruction *old_instruction) {
2253 IrInstruction *new_instruction = ir_build_handle(irb, old_instruction->scope, old_instruction->source_node);
2254 ir_link_new_instruction(new_instruction, old_instruction);
2255 return new_instruction;
2256}
2257
22432258static IrInstruction *ir_build_overflow_op(IrBuilder *irb, Scope *scope, AstNode *source_node,
22442259 IrOverflowOp op, IrInstruction *type_value, IrInstruction *op1, IrInstruction *op2,
22452260 IrInstruction *result_ptr, TypeTableEntry *result_ptr_type)
......@@ -3317,7 +3332,15 @@ static VariableTableEntry *create_local_var(CodeGen *codegen, AstNode *node, Sco
33173332static VariableTableEntry *ir_create_var(IrBuilder *irb, AstNode *node, Scope *scope, Buf *name,
33183333 bool src_is_const, bool gen_is_const, bool is_shadowable, IrInstruction *is_comptime)
33193334{
3320 VariableTableEntry *var = create_local_var(irb->codegen, node, scope, name, src_is_const, gen_is_const, is_shadowable, is_comptime);
3335 bool is_underscored = name ? buf_eql_str(name, "_") : false;
3336 VariableTableEntry *var = create_local_var( irb->codegen
3337 , node
3338 , scope
3339 , (is_underscored ? nullptr : name)
3340 , src_is_const
3341 , gen_is_const
3342 , (is_underscored ? true : is_shadowable)
3343 , is_comptime );
33213344 if (is_comptime != nullptr || gen_is_const) {
33223345 var->mem_slot_index = exec_next_mem_slot(irb->exec);
33233346 var->owner_exec = irb->exec;
......@@ -3843,6 +3866,8 @@ static IrInstruction *ir_gen_builtin_fn_call(IrBuilder *irb, Scope *scope, AstNo
38433866 return irb->codegen->invalid_instruction;
38443867 }
38453868
3869 bool is_async = exec_is_async(irb->exec);
3870
38463871 switch (builtin_fn->id) {
38473872 case BuiltinFnIdInvalid:
38483873 zig_unreachable();
......@@ -4475,6 +4500,16 @@ static IrInstruction *ir_gen_builtin_fn_call(IrBuilder *irb, Scope *scope, AstNo
44754500 return ir_lval_wrap(irb, scope, ir_build_return_address(irb, scope, node), lval);
44764501 case BuiltinFnIdFrameAddress:
44774502 return ir_lval_wrap(irb, scope, ir_build_frame_address(irb, scope, node), lval);
4503 case BuiltinFnIdHandle:
4504 if (!irb->exec->fn_entry) {
4505 add_node_error(irb->codegen, node, buf_sprintf("@handle() called outside of function definition"));
4506 return irb->codegen->invalid_instruction;
4507 }
4508 if (!is_async) {
4509 add_node_error(irb->codegen, node, buf_sprintf("@handle() in non-async function"));
4510 return irb->codegen->invalid_instruction;
4511 }
4512 return ir_lval_wrap(irb, scope, ir_build_handle(irb, scope, node), lval);
44784513 case BuiltinFnIdAlignOf:
44794514 {
44804515 AstNode *arg0_node = node->data.fn_call_expr.params.at(0);
......@@ -5159,6 +5194,11 @@ static IrInstruction *ir_gen_var_decl(IrBuilder *irb, Scope *scope, AstNode *nod
51595194
51605195 AstNodeVariableDeclaration *variable_declaration = &node->data.variable_declaration;
51615196
5197 if (buf_eql_str(variable_declaration->symbol, "_")) {
5198 add_node_error(irb->codegen, node, buf_sprintf("`_` is not a declarable symbol"));
5199 return irb->codegen->invalid_instruction;
5200 }
5201
51625202 IrInstruction *type_instruction;
51635203 if (variable_declaration->type != nullptr) {
51645204 type_instruction = ir_gen_node(irb, variable_declaration->type, scope);
......@@ -5171,6 +5211,7 @@ static IrInstruction *ir_gen_var_decl(IrBuilder *irb, Scope *scope, AstNode *nod
51715211 bool is_shadowable = false;
51725212 bool is_const = variable_declaration->is_const;
51735213 bool is_extern = variable_declaration->is_extern;
5214
51745215 IrInstruction *is_comptime = ir_build_const_bool(irb, scope, node,
51755216 ir_should_inline(irb->exec, scope) || variable_declaration->is_comptime);
51765217 VariableTableEntry *var = ir_create_var(irb, node, scope, variable_declaration->symbol,
......@@ -7069,19 +7110,8 @@ static IrInstruction *ir_gen_suspend(IrBuilder *irb, Scope *parent_scope, AstNod
70697110 if (node->data.suspend.block == nullptr) {
70707111 suspend_code = ir_build_coro_suspend(irb, parent_scope, node, nullptr, const_bool_false);
70717112 } else {
7072 assert(node->data.suspend.promise_symbol != nullptr);
7073 assert(node->data.suspend.promise_symbol->type == NodeTypeSymbol);
7074 Buf *promise_symbol_name = node->data.suspend.promise_symbol->data.symbol_expr.symbol;
70757113 Scope *child_scope;
7076 if (!buf_eql_str(promise_symbol_name, "_")) {
7077 VariableTableEntry *promise_var = ir_create_var(irb, node, parent_scope, promise_symbol_name,
7078 true, true, false, const_bool_false);
7079 ir_build_var_decl(irb, parent_scope, node, promise_var, nullptr, nullptr, irb->exec->coro_handle);
7080 child_scope = promise_var->child_scope;
7081 } else {
7082 child_scope = parent_scope;
7083 }
7084 ScopeSuspend *suspend_scope = create_suspend_scope(node, child_scope);
7114 ScopeSuspend *suspend_scope = create_suspend_scope(node, parent_scope);
70857115 suspend_scope->resume_block = resume_block;
70867116 child_scope = &suspend_scope->base;
70877117 IrInstruction *save_token = ir_build_coro_save(irb, child_scope, node, irb->exec->coro_handle);
......@@ -9598,6 +9628,9 @@ static ConstExprValue *ir_resolve_const(IrAnalyze *ira, IrInstruction *value, Un
95989628 case ConstValSpecialStatic:
95999629 return &value->value;
96009630 case ConstValSpecialRuntime:
9631 if (!type_has_bits(value->value.type)) {
9632 return &value->value;
9633 }
96019634 ir_add_error(ira, value, buf_sprintf("unable to evaluate constant expression"));
96029635 return nullptr;
96039636 case ConstValSpecialUndef:
......@@ -16099,8 +16132,14 @@ static TypeTableEntry *ir_analyze_container_init_fields_union(IrAnalyze *ira, Ir
1609916132 if (casted_field_value == ira->codegen->invalid_instruction)
1610016133 return ira->codegen->builtin_types.entry_invalid;
1610116134
16135 type_ensure_zero_bits_known(ira->codegen, casted_field_value->value.type);
16136 if (type_is_invalid(casted_field_value->value.type))
16137 return ira->codegen->builtin_types.entry_invalid;
16138
1610216139 bool is_comptime = ir_should_inline(ira->new_irb.exec, instruction->scope);
16103 if (is_comptime || casted_field_value->value.special != ConstValSpecialRuntime) {
16140 if (is_comptime || casted_field_value->value.special != ConstValSpecialRuntime ||
16141 !type_has_bits(casted_field_value->value.type))
16142 {
1610416143 ConstExprValue *field_val = ir_resolve_const(ira, casted_field_value, UndefOk);
1610516144 if (!field_val)
1610616145 return ira->codegen->builtin_types.entry_invalid;
......@@ -19007,6 +19046,14 @@ static TypeTableEntry *ir_analyze_instruction_frame_address(IrAnalyze *ira, IrIn
1900719046 return u8_ptr_const;
1900819047}
1900919048
19049static TypeTableEntry *ir_analyze_instruction_handle(IrAnalyze *ira, IrInstructionHandle *instruction) {
19050 ir_build_handle_from(&ira->new_irb, &instruction->base);
19051
19052 FnTableEntry *fn_entry = exec_fn_entry(ira->new_irb.exec);
19053 assert(fn_entry != nullptr);
19054 return get_promise_type(ira->codegen, fn_entry->type_entry->data.fn.fn_type_id.return_type);
19055}
19056
1901019057static TypeTableEntry *ir_analyze_instruction_align_of(IrAnalyze *ira, IrInstructionAlignOf *instruction) {
1901119058 IrInstruction *type_value = instruction->type_value->other;
1901219059 if (type_is_invalid(type_value->value.type))
......@@ -20982,6 +21029,8 @@ static TypeTableEntry *ir_analyze_instruction_nocast(IrAnalyze *ira, IrInstructi
2098221029 return ir_analyze_instruction_return_address(ira, (IrInstructionReturnAddress *)instruction);
2098321030 case IrInstructionIdFrameAddress:
2098421031 return ir_analyze_instruction_frame_address(ira, (IrInstructionFrameAddress *)instruction);
21032 case IrInstructionIdHandle:
21033 return ir_analyze_instruction_handle(ira, (IrInstructionHandle *)instruction);
2098521034 case IrInstructionIdAlignOf:
2098621035 return ir_analyze_instruction_align_of(ira, (IrInstructionAlignOf *)instruction);
2098721036 case IrInstructionIdOverflowOp:
......@@ -21274,6 +21323,7 @@ bool ir_has_side_effects(IrInstruction *instruction) {
2127421323 case IrInstructionIdAlignOf:
2127521324 case IrInstructionIdReturnAddress:
2127621325 case IrInstructionIdFrameAddress:
21326 case IrInstructionIdHandle:
2127721327 case IrInstructionIdTestErr:
2127821328 case IrInstructionIdUnwrapErrCode:
2127921329 case IrInstructionIdOptionalWrap:
src/ir_print.cpp+7
......@@ -791,6 +791,10 @@ static void ir_print_frame_address(IrPrint *irp, IrInstructionFrameAddress *inst
791791 fprintf(irp->f, "@frameAddress()");
792792}
793793
794static void ir_print_handle(IrPrint *irp, IrInstructionHandle *instruction) {
795 fprintf(irp->f, "@handle()");
796}
797
794798static void ir_print_return_address(IrPrint *irp, IrInstructionReturnAddress *instruction) {
795799 fprintf(irp->f, "@returnAddress()");
796800}
......@@ -1556,6 +1560,9 @@ static void ir_print_instruction(IrPrint *irp, IrInstruction *instruction) {
15561560 case IrInstructionIdFrameAddress:
15571561 ir_print_frame_address(irp, (IrInstructionFrameAddress *)instruction);
15581562 break;
1563 case IrInstructionIdHandle:
1564 ir_print_handle(irp, (IrInstructionHandle *)instruction);
1565 break;
15591566 case IrInstructionIdAlignOf:
15601567 ir_print_align_of(irp, (IrInstructionAlignOf *)instruction);
15611568 break;
src/parser.cpp+9-16
......@@ -648,12 +648,11 @@ static AstNode *ast_parse_asm_expr(ParseContext *pc, size_t *token_index, bool m
648648}
649649
650650/*
651SuspendExpression(body) = "suspend" option(("|" Symbol "|" body))
651SuspendExpression(body) = "suspend" option( body )
652652*/
653653static AstNode *ast_parse_suspend_block(ParseContext *pc, size_t *token_index, bool mandatory) {
654 size_t orig_token_index = *token_index;
655
656654 Token *suspend_token = &pc->tokens->at(*token_index);
655
657656 if (suspend_token->id == TokenIdKeywordSuspend) {
658657 *token_index += 1;
659658 } else if (mandatory) {
......@@ -663,23 +662,18 @@ static AstNode *ast_parse_suspend_block(ParseContext *pc, size_t *token_index, b
663662 return nullptr;
664663 }
665664
666 Token *bar_token = &pc->tokens->at(*token_index);
667 if (bar_token->id == TokenIdBinOr) {
668 *token_index += 1;
665 Token *lbrace = &pc->tokens->at(*token_index);
666 if (lbrace->id == TokenIdLBrace) {
667 AstNode *node = ast_create_node(pc, NodeTypeSuspend, suspend_token);
668 node->data.suspend.block = ast_parse_block(pc, token_index, true);
669 return node;
669670 } else if (mandatory) {
670 ast_expect_token(pc, suspend_token, TokenIdBinOr);
671 ast_expect_token(pc, lbrace, TokenIdLBrace);
671672 zig_unreachable();
672673 } else {
673 *token_index = orig_token_index;
674 *token_index -= 1;
674675 return nullptr;
675676 }
676
677 AstNode *node = ast_create_node(pc, NodeTypeSuspend, suspend_token);
678 node->data.suspend.promise_symbol = ast_parse_symbol(pc, token_index);
679 ast_eat_token(pc, token_index, TokenIdBinOr);
680 node->data.suspend.block = ast_parse_block(pc, token_index, true);
681
682 return node;
683677}
684678
685679/*
......@@ -3134,7 +3128,6 @@ void ast_visit_node_children(AstNode *node, void (*visit)(AstNode **, void *cont
31343128 visit_field(&node->data.await_expr.expr, visit, context);
31353129 break;
31363130 case NodeTypeSuspend:
3137 visit_field(&node->data.suspend.promise_symbol, visit, context);
31383131 visit_field(&node->data.suspend.block, visit, context);
31393132 break;
31403133 }
std/event/channel.zig+4-4
......@@ -71,10 +71,10 @@ pub fn Channel(comptime T: type) type {
7171 /// puts a data item in the channel. The promise completes when the value has been added to the
7272 /// buffer, or in the case of a zero size buffer, when the item has been retrieved by a getter.
7373 pub async fn put(self: *SelfChannel, data: T) void {
74 suspend |handle| {
74 suspend {
7575 var my_tick_node = Loop.NextTickNode{
7676 .next = undefined,
77 .data = handle,
77 .data = @handle(),
7878 };
7979 var queue_node = std.atomic.Queue(PutNode).Node{
8080 .data = PutNode{
......@@ -96,10 +96,10 @@ pub fn Channel(comptime T: type) type {
9696 // TODO integrate this function with named return values
9797 // so we can get rid of this extra result copy
9898 var result: T = undefined;
99 suspend |handle| {
99 suspend {
100100 var my_tick_node = Loop.NextTickNode{
101101 .next = undefined,
102 .data = handle,
102 .data = @handle(),
103103 };
104104 var queue_node = std.atomic.Queue(GetNode).Node{
105105 .data = GetNode{
std/event/future.zig+6-6
......@@ -100,8 +100,8 @@ test "std.event.Future" {
100100}
101101
102102async fn testFuture(loop: *Loop) void {
103 suspend |p| {
104 resume p;
103 suspend {
104 resume @handle();
105105 }
106106 var future = Future(i32).init(loop);
107107
......@@ -115,15 +115,15 @@ async fn testFuture(loop: *Loop) void {
115115}
116116
117117async fn waitOnFuture(future: *Future(i32)) i32 {
118 suspend |p| {
119 resume p;
118 suspend {
119 resume @handle();
120120 }
121121 return (await (async future.get() catch @panic("memory"))).*;
122122}
123123
124124async fn resolveFuture(future: *Future(i32)) void {
125 suspend |p| {
126 resume p;
125 suspend {
126 resume @handle();
127127 }
128128 future.data = 6;
129129 future.resolve();
std/event/group.zig+2-2
......@@ -54,10 +54,10 @@ pub fn Group(comptime ReturnType: type) type {
5454 const S = struct {
5555 async fn asyncFunc(node: **Stack.Node, args2: ...) ReturnType {
5656 // TODO this is a hack to make the memory following be inside the coro frame
57 suspend |p| {
57 suspend {
5858 var my_node: Stack.Node = undefined;
5959 node.* = &my_node;
60 resume p;
60 resume @handle();
6161 }
6262
6363 // TODO this allocation elision should be guaranteed because we await it in
std/event/lock.zig+6-29
......@@ -90,10 +90,10 @@ pub const Lock = struct {
9090 }
9191
9292 pub async fn acquire(self: *Lock) Held {
93 suspend |handle| {
93 suspend {
9494 // TODO explicitly put this memory in the coroutine frame #1194
9595 var my_tick_node = Loop.NextTickNode{
96 .data = handle,
96 .data = @handle(),
9797 .next = undefined,
9898 };
9999
......@@ -106,35 +106,12 @@ pub const Lock = struct {
106106 // will attempt to grab the lock.
107107 _ = @atomicRmw(u8, &self.queue_empty_bit, AtomicRmwOp.Xchg, 0, AtomicOrder.SeqCst);
108108
109 while (true) {
110 const old_bit = @atomicRmw(u8, &self.shared_bit, AtomicRmwOp.Xchg, 1, AtomicOrder.SeqCst);
111 if (old_bit != 0) {
112 // We did not obtain the lock. Trust that our queue entry will resume us, and allow
113 // suspend to complete.
114 break;
115 }
116 // We got the lock. However we might have already been resumed from the queue.
109 const old_bit = @atomicRmw(u8, &self.shared_bit, AtomicRmwOp.Xchg, 1, AtomicOrder.SeqCst);
110 if (old_bit == 0) {
117111 if (self.queue.get()) |node| {
118112 // Whether this node is us or someone else, we tail resume it.
119113 resume node.data;
120 break;
121 } else {
122 // We already got resumed, and there are none left in the queue, which means that
123 // we aren't even supposed to hold the lock right now.
124 _ = @atomicRmw(u8, &self.queue_empty_bit, AtomicRmwOp.Xchg, 1, AtomicOrder.SeqCst);
125 _ = @atomicRmw(u8, &self.shared_bit, AtomicRmwOp.Xchg, 0, AtomicOrder.SeqCst);
126
127 // There might be a queue item. If we know the queue is empty, we can be done,
128 // because the other actor will try to obtain the lock.
129 // But if there's a queue item, we are the actor which must loop and attempt
130 // to grab the lock again.
131 if (@atomicLoad(u8, &self.queue_empty_bit, AtomicOrder.SeqCst) == 1) {
132 break;
133 } else {
134 continue;
135 }
136114 }
137 unreachable;
138115 }
139116 }
140117
......@@ -164,8 +141,8 @@ test "std.event.Lock" {
164141
165142async fn testLock(loop: *Loop, lock: *Lock) void {
166143 // TODO explicitly put next tick node memory in the coroutine frame #1194
167 suspend |p| {
168 resume p;
144 suspend {
145 resume @handle();
169146 }
170147 const handle1 = async lockRunner(lock) catch @panic("out of memory");
171148 var tick_node1 = Loop.NextTickNode{
std/event/loop.zig+7-7
......@@ -331,11 +331,11 @@ pub const Loop = struct {
331331
332332 pub async fn waitFd(self: *Loop, fd: i32) !void {
333333 defer self.removeFd(fd);
334 suspend |p| {
334 suspend {
335335 // TODO explicitly put this memory in the coroutine frame #1194
336336 var resume_node = ResumeNode{
337337 .id = ResumeNode.Id.Basic,
338 .handle = p,
338 .handle = @handle(),
339339 };
340340 try self.addFd(fd, &resume_node);
341341 }
......@@ -417,11 +417,11 @@ pub const Loop = struct {
417417 pub fn call(self: *Loop, comptime func: var, args: ...) !(promise->@typeOf(func).ReturnType) {
418418 const S = struct {
419419 async fn asyncFunc(loop: *Loop, handle: *promise->@typeOf(func).ReturnType, args2: ...) @typeOf(func).ReturnType {
420 suspend |p| {
421 handle.* = p;
420 suspend {
421 handle.* = @handle();
422422 var my_tick_node = Loop.NextTickNode{
423423 .next = undefined,
424 .data = p,
424 .data = @handle(),
425425 };
426426 loop.onNextTick(&my_tick_node);
427427 }
......@@ -439,10 +439,10 @@ pub const Loop = struct {
439439 /// CPU bound tasks would be waiting in the event loop but never get started because no async I/O
440440 /// is performed.
441441 pub async fn yield(self: *Loop) void {
442 suspend |p| {
442 suspend {
443443 var my_tick_node = Loop.NextTickNode{
444444 .next = undefined,
445 .data = p,
445 .data = @handle(),
446446 };
447447 self.onNextTick(&my_tick_node);
448448 }
std/event/tcp.zig+4-4
......@@ -88,8 +88,8 @@ pub const Server = struct {
8888 },
8989 error.ProcessFdQuotaExceeded => {
9090 errdefer std.os.emfile_promise_queue.remove(&self.waiting_for_emfile_node);
91 suspend |p| {
92 self.waiting_for_emfile_node = PromiseNode.init(p);
91 suspend {
92 self.waiting_for_emfile_node = PromiseNode.init( @handle() );
9393 std.os.emfile_promise_queue.append(&self.waiting_for_emfile_node);
9494 }
9595 continue;
......@@ -141,8 +141,8 @@ test "listen on a port, send bytes, receive bytes" {
141141 (await next_handler) catch |err| {
142142 std.debug.panic("unable to handle connection: {}\n", err);
143143 };
144 suspend |p| {
145 cancel p;
144 suspend {
145 cancel @handle();
146146 }
147147 }
148148 async fn errorableHandler(self: *Self, _addr: *const std.net.Address, _socket: *const std.os.File) !void {
std/fmt/index.zig+9
......@@ -248,6 +248,11 @@ pub fn formatIntValue(
248248 return formatAsciiChar(value, context, Errors, output);
249249 }
250250 },
251 'b' => {
252 radix = 2;
253 uppercase = false;
254 width = 0;
255 },
251256 'd' => {
252257 radix = 10;
253258 uppercase = false;
......@@ -874,6 +879,10 @@ test "fmt.format" {
874879 const value: u8 = 'a';
875880 try testFmt("u8: a\n", "u8: {c}\n", value);
876881 }
882 {
883 const value: u8 = 0b1100;
884 try testFmt("u8: 0b1100\n", "u8: 0b{b}\n", value);
885 }
877886 {
878887 const value: [3]u8 = "abc";
879888 try testFmt("array: abc\n", "array: {}\n", value);
std/os/index.zig+13-13
......@@ -130,16 +130,10 @@ pub fn getRandomBytes(buf: []u8) !void {
130130 try posixRead(fd, buf);
131131 },
132132 Os.windows => {
133 var hCryptProv: windows.HCRYPTPROV = undefined;
134 if (windows.CryptAcquireContextA(&hCryptProv, null, null, windows.PROV_RSA_FULL, 0) == 0) {
135 const err = windows.GetLastError();
136 return switch (err) {
137 else => unexpectedErrorWindows(err),
138 };
139 }
140 defer _ = windows.CryptReleaseContext(hCryptProv, 0);
141
142 if (windows.CryptGenRandom(hCryptProv, @intCast(windows.DWORD, buf.len), buf.ptr) == 0) {
133 // Call RtlGenRandom() instead of CryptGetRandom() on Windows
134 // https://github.com/rust-lang-nursery/rand/issues/111
135 // https://bugzilla.mozilla.org/show_bug.cgi?id=504270
136 if (windows.RtlGenRandom(buf.ptr, buf.len) == 0) {
143137 const err = windows.GetLastError();
144138 return switch (err) {
145139 else => unexpectedErrorWindows(err),
......@@ -159,8 +153,14 @@ pub fn getRandomBytes(buf: []u8) !void {
159153}
160154
161155test "os.getRandomBytes" {
162 var buf: [50]u8 = undefined;
163 try getRandomBytes(buf[0..]);
156 var buf_a: [50]u8 = undefined;
157 var buf_b: [50]u8 = undefined;
158 // Call Twice
159 try getRandomBytes(buf_a[0..]);
160 try getRandomBytes(buf_b[0..]);
161
162 // Check if random (not 100% conclusive)
163 assert( !mem.eql(u8, buf_a, buf_b) );
164164}
165165
166166/// Raises a signal in the current kernel thread, ending its execution.
......@@ -2790,7 +2790,7 @@ pub fn cpuCount(fallback_allocator: *mem.Allocator) CpuCountError!usize {
27902790 builtin.Os.macosx => {
27912791 var count: c_int = undefined;
27922792 var count_len: usize = @sizeOf(c_int);
2793 const rc = posix.sysctlbyname(c"hw.ncpu", @ptrCast(*c_void, &count), &count_len, null, 0);
2793 const rc = posix.sysctlbyname(c"hw.logicalcpu", @ptrCast(*c_void, &count), &count_len, null, 0);
27942794 const err = posix.getErrno(rc);
27952795 switch (err) {
27962796 0 => return @intCast(usize, count),
std/os/linux/index.zig+1-1
......@@ -944,7 +944,7 @@ pub fn setgroups(size: usize, list: *const u32) usize {
944944}
945945
946946pub fn getpid() i32 {
947 return @bitCast(i32, u32(syscall0(SYS_getpid)));
947 return @bitCast(i32, @truncate(u32, syscall0(SYS_getpid)));
948948}
949949
950950pub fn sigprocmask(flags: u32, noalias set: *const sigset_t, noalias oldset: ?*sigset_t) usize {
std/os/linux/test.zig+4
......@@ -3,6 +3,10 @@ const builtin = @import("builtin");
33const linux = std.os.linux;
44const assert = std.debug.assert;
55
6test "getpid" {
7 assert(linux.getpid() != 0);
8}
9
610test "timer" {
711 const epoll_fd = linux.epoll_create();
812 var err = linux.getErrno(epoll_fd);
std/os/windows/advapi32.zig+5
......@@ -28,3 +28,8 @@ pub extern "advapi32" stdcallcc fn RegOpenKeyExW(hKey: HKEY, lpSubKey: LPCWSTR,
2828
2929pub extern "advapi32" stdcallcc fn RegQueryValueExW(hKey: HKEY, lpValueName: LPCWSTR, lpReserved: LPDWORD,
3030 lpType: LPDWORD, lpData: LPBYTE, lpcbData: LPDWORD,) LSTATUS;
31
32// RtlGenRandom is known as SystemFunction036 under advapi32
33// http://msdn.microsoft.com/en-us/library/windows/desktop/aa387694.aspx */
34pub extern "advapi32" stdcallcc fn SystemFunction036(output: [*]u8, length: usize) BOOL;
35pub const RtlGenRandom = SystemFunction036;
std/os/windows/util.zig+1-1
......@@ -166,7 +166,7 @@ pub fn windowsUnloadDll(hModule: windows.HMODULE) void {
166166}
167167
168168test "InvalidDll" {
169 if (builtin.os != builtin.Os.windows) return;
169 if (builtin.os != builtin.Os.windows) return error.SkipZigTest;
170170
171171 const DllName = "asdf.dll";
172172 const allocator = std.debug.global_allocator;
std/rand/index.zig+5-4
......@@ -30,7 +30,7 @@ pub const DefaultCsprng = Isaac64;
3030pub const Random = struct {
3131 fillFn: fn (r: *Random, buf: []u8) void,
3232
33 /// Read random bytes into the specified buffer until fill.
33 /// Read random bytes into the specified buffer until full.
3434 pub fn bytes(r: *Random, buf: []u8) void {
3535 r.fillFn(r, buf);
3636 }
......@@ -48,10 +48,10 @@ pub const Random = struct {
4848 }
4949 }
5050
51 /// Get a random unsigned integer with even distribution between `start`
52 /// inclusive and `end` exclusive.
51 /// Return a random integer with even distribution between `start`
52 /// inclusive and `end` exclusive. `start` must be less than `end`.
5353 pub fn range(r: *Random, comptime T: type, start: T, end: T) T {
54 assert(start <= end);
54 assert(start < end);
5555 if (T.is_signed) {
5656 const uint = @IntType(false, T.bit_count);
5757 if (start >= 0 and end >= 0) {
......@@ -664,6 +664,7 @@ test "Random range" {
664664 testRange(&prng.random, -4, 3);
665665 testRange(&prng.random, -4, -1);
666666 testRange(&prng.random, 10, 14);
667 // TODO: test that prng.random.range(1, 1) causes an assertion error
667668}
668669
669670fn testRange(r: *Random, start: i32, end: i32) void {
std/zig/ast.zig-12
......@@ -1778,19 +1778,12 @@ pub const Node = struct {
17781778
17791779 pub const Suspend = struct {
17801780 base: Node,
1781 label: ?TokenIndex,
17821781 suspend_token: TokenIndex,
1783 payload: ?*Node,
17841782 body: ?*Node,
17851783
17861784 pub fn iterate(self: *Suspend, index: usize) ?*Node {
17871785 var i = index;
17881786
1789 if (self.payload) |payload| {
1790 if (i < 1) return payload;
1791 i -= 1;
1792 }
1793
17941787 if (self.body) |body| {
17951788 if (i < 1) return body;
17961789 i -= 1;
......@@ -1800,7 +1793,6 @@ pub const Node = struct {
18001793 }
18011794
18021795 pub fn firstToken(self: *Suspend) TokenIndex {
1803 if (self.label) |label| return label;
18041796 return self.suspend_token;
18051797 }
18061798
......@@ -1809,10 +1801,6 @@ pub const Node = struct {
18091801 return body.lastToken();
18101802 }
18111803
1812 if (self.payload) |payload| {
1813 return payload.lastToken();
1814 }
1815
18161804 return self.suspend_token;
18171805 }
18181806 };
std/zig/parse.zig+14-19
......@@ -852,19 +852,6 @@ pub fn parse(allocator: *mem.Allocator, source: []const u8) !ast.Tree {
852852 }) catch unreachable;
853853 continue;
854854 },
855 Token.Id.Keyword_suspend => {
856 const node = try arena.create(ast.Node.Suspend{
857 .base = ast.Node{ .id = ast.Node.Id.Suspend },
858 .label = ctx.label,
859 .suspend_token = token_index,
860 .payload = null,
861 .body = null,
862 });
863 ctx.opt_ctx.store(&node.base);
864 stack.append(State{ .SuspendBody = node }) catch unreachable;
865 try stack.append(State{ .Payload = OptionalCtx{ .Optional = &node.payload } });
866 continue;
867 },
868855 Token.Id.Keyword_inline => {
869856 stack.append(State{
870857 .Inline = InlineCtx{
......@@ -1415,10 +1402,21 @@ pub fn parse(allocator: *mem.Allocator, source: []const u8) !ast.Tree {
14151402 },
14161403
14171404 State.SuspendBody => |suspend_node| {
1418 if (suspend_node.payload != null) {
1419 try stack.append(State{ .AssignmentExpressionBegin = OptionalCtx{ .RequiredNull = &suspend_node.body } });
1405 const token = nextToken(&tok_it, &tree);
1406 switch (token.ptr.id) {
1407 Token.Id.Semicolon => {
1408 prevToken(&tok_it, &tree);
1409 continue;
1410 },
1411 Token.Id.LBrace => {
1412 prevToken(&tok_it, &tree);
1413 try stack.append(State{ .AssignmentExpressionBegin = OptionalCtx{ .RequiredNull = &suspend_node.body } });
1414 continue;
1415 },
1416 else => {
1417 ((try tree.errors.addOne())).* = Error{ .InvalidToken = Error.InvalidToken{ .token = token.index } };
1418 },
14201419 }
1421 continue;
14221420 },
14231421 State.AsyncAllocator => |async_node| {
14241422 if (eatToken(&tok_it, &tree, Token.Id.AngleBracketLeft) == null) {
......@@ -3086,15 +3084,12 @@ fn parseBlockExpr(stack: *std.ArrayList(State), arena: *mem.Allocator, ctx: *con
30863084 Token.Id.Keyword_suspend => {
30873085 const node = try arena.create(ast.Node.Suspend{
30883086 .base = ast.Node{ .id = ast.Node.Id.Suspend },
3089 .label = null,
30903087 .suspend_token = token_index,
3091 .payload = null,
30923088 .body = null,
30933089 });
30943090 ctx.store(&node.base);
30953091
30963092 stack.append(State{ .SuspendBody = node }) catch unreachable;
3097 try stack.append(State{ .Payload = OptionalCtx{ .Optional = &node.payload } });
30983093 return true;
30993094 },
31003095 Token.Id.Keyword_if => {
std/zig/parser_test.zig+4-4
......@@ -898,11 +898,11 @@ test "zig fmt: union(enum(u32)) with assigned enum values" {
898898 );
899899}
900900
901test "zig fmt: labeled suspend" {
901test "zig fmt: resume from suspend block" {
902902 try testCanonical(
903903 \\fn foo() void {
904 \\ s: suspend |p| {
905 \\ break :s;
904 \\ suspend {
905 \\ resume @handle();
906906 \\ }
907907 \\}
908908 \\
......@@ -1784,7 +1784,7 @@ test "zig fmt: coroutines" {
17841784 \\ x += 1;
17851785 \\ suspend;
17861786 \\ x += 1;
1787 \\ suspend |p| {}
1787 \\ suspend;
17881788 \\ const p: promise->void = async simpleAsyncFn() catch unreachable;
17891789 \\ await p;
17901790 \\}
std/zig/render.zig+1-15
......@@ -323,21 +323,7 @@ fn renderExpression(
323323 ast.Node.Id.Suspend => {
324324 const suspend_node = @fieldParentPtr(ast.Node.Suspend, "base", base);
325325
326 if (suspend_node.label) |label| {
327 try renderToken(tree, stream, label, indent, start_col, Space.None);
328 try renderToken(tree, stream, tree.nextToken(label), indent, start_col, Space.Space);
329 }
330
331 if (suspend_node.payload) |payload| {
332 if (suspend_node.body) |body| {
333 try renderToken(tree, stream, suspend_node.suspend_token, indent, start_col, Space.Space);
334 try renderExpression(allocator, stream, tree, indent, start_col, payload, Space.Space);
335 return renderExpression(allocator, stream, tree, indent, start_col, body, space);
336 } else {
337 try renderToken(tree, stream, suspend_node.suspend_token, indent, start_col, Space.Space);
338 return renderExpression(allocator, stream, tree, indent, start_col, payload, space);
339 }
340 } else if (suspend_node.body) |body| {
326 if (suspend_node.body) |body| {
341327 try renderToken(tree, stream, suspend_node.suspend_token, indent, start_col, Space.Space);
342328 return renderExpression(allocator, stream, tree, indent, start_col, body, space);
343329 } else {
test/behavior.zig+1
......@@ -60,6 +60,7 @@ comptime {
6060 _ = @import("cases/try.zig");
6161 _ = @import("cases/type_info.zig");
6262 _ = @import("cases/undefined.zig");
63 _ = @import("cases/underscore.zig");
6364 _ = @import("cases/union.zig");
6465 _ = @import("cases/var_args.zig");
6566 _ = @import("cases/void.zig");
test/cases/cancel.zig+2-2
......@@ -85,8 +85,8 @@ async fn b4() void {
8585 defer {
8686 defer_b4 = true;
8787 }
88 suspend |p| {
89 b4_handle = p;
88 suspend {
89 b4_handle = @handle();
9090 }
9191 suspend;
9292}
test/cases/coroutine_await_struct.zig+2-2
......@@ -30,9 +30,9 @@ async fn await_amain() void {
3030}
3131async fn await_another() Foo {
3232 await_seq('c');
33 suspend |p| {
33 suspend {
3434 await_seq('d');
35 await_a_promise = p;
35 await_a_promise = @handle();
3636 }
3737 await_seq('g');
3838 return Foo{ .x = 1234 };
test/cases/coroutines.zig+12-7
......@@ -62,10 +62,15 @@ test "coroutine suspend with block" {
6262var a_promise: promise = undefined;
6363var result = false;
6464async fn testSuspendBlock() void {
65 suspend |p| {
66 comptime assert(@typeOf(p) == promise->void);
67 a_promise = p;
65 suspend {
66 comptime assert(@typeOf(@handle()) == promise->void);
67 a_promise = @handle();
6868 }
69
70 //Test to make sure that @handle() works as advertised (issue #1296)
71 //var our_handle: promise = @handle();
72 assert( a_promise == @handle() );
73
6974 result = true;
7075}
7176
......@@ -93,9 +98,9 @@ async fn await_amain() void {
9398}
9499async fn await_another() i32 {
95100 await_seq('c');
96 suspend |p| {
101 suspend {
97102 await_seq('d');
98 await_a_promise = p;
103 await_a_promise = @handle();
99104 }
100105 await_seq('g');
101106 return 1234;
......@@ -244,8 +249,8 @@ test "break from suspend" {
244249 std.debug.assert(my_result == 2);
245250}
246251async fn testBreakFromSuspend(my_result: *i32) void {
247 suspend |p| {
248 resume p;
252 suspend {
253 resume @handle();
249254 }
250255 my_result.* += 1;
251256 suspend;
test/cases/underscore.zig created+28
......@@ -0,0 +1,28 @@
1const std = @import("std");
2const assert = std.debug.assert;
3
4test "ignore lval with underscore" {
5 _ = false;
6}
7
8test "ignore lval with underscore (for loop)" {
9 for ([]void{}) |_, i| {
10 for ([]void{}) |_, j| {
11 break;
12 }
13 break;
14 }
15}
16
17test "ignore lval with underscore (while loop)" {
18 while (optionalReturnError()) |_| {
19 while (optionalReturnError()) |_| {
20 break;
21 } else |_| { }
22 break;
23 } else |_| { }
24}
25
26fn optionalReturnError() !?u32 {
27 return error.optionalReturnError;
28}
test/cases/union.zig+14
......@@ -297,3 +297,17 @@ test "access a member of tagged union with conflicting enum tag name" {
297297
298298 comptime assert(Bar.A == u8);
299299}
300
301test "tagged union initialization with runtime void" {
302 assert(testTaggedUnionInit({}));
303}
304
305const TaggedUnionWithAVoid = union(enum) {
306 A,
307 B: i32,
308};
309
310fn testTaggedUnionInit(x: var) bool {
311 const y = TaggedUnionWithAVoid{ .A = x };
312 return @TagType(TaggedUnionWithAVoid)(y) == TaggedUnionWithAVoid.A;
313}
test/compile_errors.zig+81-2
......@@ -1,6 +1,85 @@
11const tests = @import("tests.zig");
22
33pub fn addCases(cases: *tests.CompileErrorContext) void {
4 cases.add(
5 "@handle() called outside of function definition",
6 \\var handle_undef: promise = undefined;
7 \\var handle_dummy: promise = @handle();
8 \\export fn entry() bool {
9 \\ return handle_undef == handle_dummy;
10 \\}
11 ,
12 ".tmp_source.zig:2:29: error: @handle() called outside of function definition",
13 );
14
15 cases.add(
16 "@handle() in non-async function",
17 \\export fn entry() bool {
18 \\ var handle_undef: promise = undefined;
19 \\ return handle_undef == @handle();
20 \\}
21 ,
22 ".tmp_source.zig:3:28: error: @handle() in non-async function",
23 );
24
25 cases.add(
26 "`_` is not a declarable symbol",
27 \\export fn f1() usize {
28 \\ var _: usize = 2;
29 \\ return _;
30 \\}
31 ,
32 ".tmp_source.zig:2:5: error: `_` is not a declarable symbol",
33 ".tmp_source.zig:3:12: error: use of undeclared identifier '_'",
34 );
35
36 cases.add(
37 "`_` should not be usable inside for",
38 \\export fn returns() void {
39 \\ for ([]void{}) |_, i| {
40 \\ for ([]void{}) |_, j| {
41 \\ return _;
42 \\ }
43 \\ }
44 \\}
45 ,
46 ".tmp_source.zig:4:20: error: use of undeclared identifier '_'",
47 );
48
49 cases.add(
50 "`_` should not be usable inside while",
51 \\export fn returns() void {
52 \\ while (optionalReturn()) |_| {
53 \\ while (optionalReturn()) |_| {
54 \\ return _;
55 \\ }
56 \\ }
57 \\}
58 \\fn optionalReturn() ?u32 {
59 \\ return 1;
60 \\}
61 ,
62 ".tmp_source.zig:4:20: error: use of undeclared identifier '_'",
63 );
64
65 cases.add(
66 "`_` should not be usable inside while else",
67 \\export fn returns() void {
68 \\ while (optionalReturnError()) |_| {
69 \\ while (optionalReturnError()) |_| {
70 \\ return;
71 \\ } else |_| {
72 \\ if (_ == error.optionalReturnError) return;
73 \\ }
74 \\ }
75 \\}
76 \\fn optionalReturnError() !?u32 {
77 \\ return error.optionalReturnError;
78 \\}
79 ,
80 ".tmp_source.zig:6:17: error: use of undeclared identifier '_'",
81 );
82
483 cases.add(
584 "while loop body expression ignored",
685 \\fn returns() usize {
......@@ -367,8 +446,8 @@ pub fn addCases(cases: *tests.CompileErrorContext) void {
367446 \\}
368447 \\
369448 \\async fn foo() void {
370 \\ suspend |p| {
371 \\ suspend |p1| {
449 \\ suspend {
450 \\ suspend {
372451 \\ }
373452 \\ }
374453 \\}