diff options
| author | Andrew Kelley <andrew@ziglang.org> | 2021-03-31 18:05:37 -0700 |
|---|---|---|
| committer | Andrew Kelley <andrew@ziglang.org> | 2021-03-31 18:05:37 -0700 |
| commit | 3cebaaad1ca16a9e0203ed8c1684d0ce72da9487 (patch) | |
| tree | a153e50f75fad0a1b6c683ad5c06e28e102ff848 /src/Module.zig | |
| parent | 08eedc962d8e2582db8fb5b4a50114f2913519fd (diff) | |
| download | zig-3cebaaad1ca16a9e0203ed8c1684d0ce72da9487.tar.gz zig-3cebaaad1ca16a9e0203ed8c1684d0ce72da9487.zip | |
astgen: improved handling of coercion
GenZir struct now has rl_ty_inst field which tracks the result location
type (if any) a block expects all of its results to be coerced to.
Remove a redundant coercion on const local initialization with a
specified type.
Switch expressions, during elision of store_to_block_ptr instructions,
now re-purpose them to be type coercion when the block has a type in the
result location.
Diffstat (limited to 'src/Module.zig')
| -rw-r--r-- | src/Module.zig | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/Module.zig b/src/Module.zig index 91790aa1b6..bceaf43425 100644 --- a/src/Module.zig +++ b/src/Module.zig @@ -942,6 +942,8 @@ pub const Scope = struct { break_result_loc: AstGen.ResultLoc = undefined, /// When a block has a pointer result location, here it is. rl_ptr: zir.Inst.Ref = .none, + /// When a block has a type result location, here it is. + rl_ty_inst: zir.Inst.Ref = .none, /// Keeps track of how many branches of a block did not actually /// consume the result location. astgen uses this to figure out /// whether to rely on break instructions or writing to the result @@ -1001,7 +1003,11 @@ pub const Scope = struct { // we emit ZIR for the block break instructions to have the result values, // and then rvalue() on that to pass the value to the result location. switch (parent_rl) { - .discard, .none, .ty, .ptr, .ref => { + .ty => |ty_inst| { + gz.rl_ty_inst = ty_inst; + gz.break_result_loc = parent_rl; + }, + .discard, .none, .ptr, .ref => { gz.break_result_loc = parent_rl; }, @@ -1016,6 +1022,7 @@ pub const Scope = struct { }, .block_ptr => |parent_block_scope| { + gz.rl_ty_inst = parent_block_scope.rl_ty_inst; gz.rl_ptr = parent_block_scope.rl_ptr; gz.break_result_loc = .{ .block_ptr = gz }; }, |
