Commit e6a71e9e7a6c9708f568bc8a024e91bdd027cae9 | zig

author	Alex Rønne Petersen <alex@alexrp.com> 2025-04-14 18:40:47+02:00
committer	Alex Rønne Petersen <alex@alexrp.com> 2025-04-28 00:58:53+02:00
log	e6a71e9e7a6c9708f568bc8a024e91bdd027cae9
tree	9ed2d25dd4e59fdfcce84c3627150ce8cce30187
parent	168981c67825fbdb3b70da8ff9f0f3b691bd36e5
signature	Signed by SSH key SHA256:7B/LJ7bpR1eX8aCXSr4mtd5M45VMPKcx9zY8e95b5QM

Sema: Fix some ptr alignment checks to handle a potential ISA tag bit.

Closes #23570.

2 files changed, 26 insertions(+), 3 deletions(-)

src
- Sema.zig
test
- behavior
  - align.zig

src/Sema.zig+16-3

 @@ -23235,8 +23235,16 @@ fn ptrFromIntVal(
     const addr = try operand_val.toUnsignedIntSema(pt);
     if (!ptr_ty.isAllowzeroPtr(zcu) and addr == 0)
         return sema.fail(block, operand_src, "pointer type '{}' does not allow address zero", .{ptr_ty.fmt(pt)});
-    if (addr != 0 and ptr_align != .none and !ptr_align.check(addr))
+    if (addr != 0 and ptr_align != .none) {
-        return sema.fail(block, operand_src, "pointer type '{}' requires aligned address", .{ptr_ty.fmt(pt)});
+        const masked_addr = if (ptr_ty.childType(zcu).fnPtrMaskOrNull(zcu)) |mask|
+            addr & mask
+        else
+            addr;
+        if (!ptr_align.check(masked_addr)) {
+            return sema.fail(block, operand_src, "pointer type '{}' requires aligned address", .{ptr_ty.fmt(pt)});
+        }
+    }
     return switch (ptr_ty.zigTypeTag(zcu)) {
         .optional => Value.fromInterned(try pt.intern(.{ .opt = .{
 @@ -23752,7 +23760,12 @@ fn ptrCastFull(
         if (dest_align.compare(.gt, src_align)) {
             if (try ptr_val.getUnsignedIntSema(pt)) |addr| {
-                if (!dest_align.check(addr)) {
+                const masked_addr = if (Type.fromInterned(dest_info.child).fnPtrMaskOrNull(zcu)) |mask|
+                    addr & mask
+                else
+                    addr;
+                if (!dest_align.check(masked_addr)) {
                     return sema.fail(block, operand_src, "pointer address 0x{X} is not aligned to {d} bytes", .{
                         addr,
                         dest_align.toByteUnits().?,

test/behavior/align.zig+10

...	@@ -635,3 +635,13 @@ test "function pointer @intFromPtr/@ptrFromInt roundtrip" {	...	@@ -635,3 +635,13 @@ test "function pointer @intFromPtr/@ptrFromInt roundtrip" {
635		635
636	try std.testing.expectEqual(nothing_ptr, nothing_ptr2);	636	try std.testing.expectEqual(nothing_ptr, nothing_ptr2);
637	}	637	}
		638
		639	test "function pointer align mask" {
		640	if (!(builtin.cpu.arch.isArm() or builtin.cpu.arch.isMIPS())) return error.SkipZigTest;
		641
		642	const a: *const fn () callconv(.c) void = @ptrFromInt(0x20202021);
		643	_ = &a;
		644
		645	const b: *align(16) const fn () callconv(.c) void = @alignCast(a);
		646	_ = &b;
		647	}