aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authormlugg <mlugg@mlugg.co.uk>2024-09-15 15:45:20 +0100
committermlugg <mlugg@mlugg.co.uk>2024-09-16 16:42:42 +0100
commit1365be5d02429dbbbfab43133d399b2af42047c5 (patch)
tree2cb61dabb88b84094701df907ad67479951f53a8 /src
parent5d7fa5513f92a43a418e3c5c4d27f0b61db313ff (diff)
downloadzig-1365be5d02429dbbbfab43133d399b2af42047c5.tar.gz
zig-1365be5d02429dbbbfab43133d399b2af42047c5.zip
compiler: provide correct result types to `+=` and `-=`
Resolves: #21341
Diffstat (limited to 'src')
-rw-r--r--src/Sema.zig28
-rw-r--r--src/print_zir.zig7
2 files changed, 35 insertions, 0 deletions
diff --git a/src/Sema.zig b/src/Sema.zig
index 7a4b9b3210..b859b8a183 100644
--- a/src/Sema.zig
+++ b/src/Sema.zig
@@ -1361,6 +1361,7 @@ fn analyzeBodyInner(
.value_placeholder => unreachable, // never appears in a body
.field_parent_ptr => try sema.zirFieldParentPtr(block, extended),
.builtin_value => try sema.zirBuiltinValue(extended),
+ .inplace_arith_result_ty => try sema.zirInplaceArithResultTy(extended),
};
},
@@ -27342,6 +27343,33 @@ fn zirBuiltinValue(sema: *Sema, extended: Zir.Inst.Extended.InstData) CompileErr
return Air.internedToRef(ty.toIntern());
}
+fn zirInplaceArithResultTy(sema: *Sema, extended: Zir.Inst.Extended.InstData) CompileError!Air.Inst.Ref {
+ const pt = sema.pt;
+ const zcu = pt.zcu;
+
+ const lhs = try sema.resolveInst(@enumFromInt(extended.operand));
+ const lhs_ty = sema.typeOf(lhs);
+
+ const op: Zir.Inst.InplaceOp = @enumFromInt(extended.small);
+ const ty: Type = switch (op) {
+ .add_eq => ty: {
+ const ptr_size = lhs_ty.ptrSizeOrNull(zcu) orelse break :ty lhs_ty;
+ switch (ptr_size) {
+ .One, .Slice => break :ty lhs_ty, // invalid, let it error
+ .Many, .C => break :ty .usize, // `[*]T + usize`
+ }
+ },
+ .sub_eq => ty: {
+ const ptr_size = lhs_ty.ptrSizeOrNull(zcu) orelse break :ty lhs_ty;
+ switch (ptr_size) {
+ .One, .Slice => break :ty lhs_ty, // invalid, let it error
+ .Many, .C => break :ty .generic_poison, // could be `[*]T - [*]T` or `[*]T - usize`
+ }
+ },
+ };
+ return Air.internedToRef(ty.toIntern());
+}
+
fn zirBranchHint(sema: *Sema, block: *Block, extended: Zir.Inst.Extended.InstData) CompileError!void {
const pt = sema.pt;
const zcu = pt.zcu;
diff --git a/src/print_zir.zig b/src/print_zir.zig
index f5c83c98e2..c5f1517edd 100644
--- a/src/print_zir.zig
+++ b/src/print_zir.zig
@@ -620,6 +620,7 @@ const Writer = struct {
.closure_get => try self.writeClosureGet(stream, extended),
.field_parent_ptr => try self.writeFieldParentPtr(stream, extended),
.builtin_value => try self.writeBuiltinValue(stream, extended),
+ .inplace_arith_result_ty => try self.writeInplaceArithResultTy(stream, extended),
}
}
@@ -2781,6 +2782,12 @@ const Writer = struct {
try self.writeSrcNode(stream, @bitCast(extended.operand));
}
+ fn writeInplaceArithResultTy(self: *Writer, stream: anytype, extended: Zir.Inst.Extended.InstData) !void {
+ const op: Zir.Inst.InplaceOp = @enumFromInt(extended.small);
+ try self.writeInstRef(stream, @enumFromInt(extended.operand));
+ try stream.print(", {s}))", .{@tagName(op)});
+ }
+
fn writeInstRef(self: *Writer, stream: anytype, ref: Zir.Inst.Ref) !void {
if (ref == .none) {
return stream.writeAll(".none");