zig

commit 804740af4ced8389e21df31a908c4212e32a477a (tree)
parent 5580a69d714af92883e2031a131c30917162dc15
Author: Andrew Kelley <andrew@ziglang.org>
Date:   Mon, 29 May 2023 13:47:13 -0700

InternPool: avoid indexToKey recursion for type_slice

This is a hot function, and recursion makes it more difficult to
profile, as well as likely making it more difficult to optimize.

Diffstat:
M
src/InternPool.zig
 | 
19
+++++++++++++++----

1 file changed, 15 insertions(+), 4 deletions(-)
diff --git a/src/InternPool.zig b/src/InternPool.zig
@@ -2455,10 +2455,21 @@ pub fn indexToKey(ip: *const InternPool, index: Index) Key {
         },
 
         .type_slice => {
-            const ptr_type_index = @intToEnum(Index, data);
-            var result = ip.indexToKey(ptr_type_index).ptr_type;
-            result.size = .Slice;
-            return .{ .ptr_type = result };
+            assert(ip.items.items(.tag)[data] == .type_pointer);
+            const ptr_info = ip.extraData(Pointer, ip.items.items(.data)[data]);
+            return .{ .ptr_type = .{
+                .elem_type = ptr_info.child,
+                .sentinel = ptr_info.sentinel,
+                .alignment = ptr_info.flags.alignment,
+                .size = .Slice,
+                .is_const = ptr_info.flags.is_const,
+                .is_volatile = ptr_info.flags.is_volatile,
+                .is_allowzero = ptr_info.flags.is_allowzero,
+                .address_space = ptr_info.flags.address_space,
+                .vector_index = ptr_info.flags.vector_index,
+                .host_size = ptr_info.packed_offset.host_size,
+                .bit_offset = ptr_info.packed_offset.bit_offset,
+            } };
         },
 
         .type_optional => .{ .opt_type = @intToEnum(Index, data) },