update to latest zig

build.zig

@@ -7,15 +7,19 @@ pub fn build(b: *std.Build) void {
     const maybe_video = b.option(std.Build.LazyPath, "video", "Set the path to the Vulkan Video registry (video.xml)");
     const test_step = b.step("test", "Run all the tests");
 
+    const root_module = b.createModule(.{
+        .root_source_file = b.path("src/main.zig"),
+        .target = target,
+        .optimize = optimize,
+    });
+
     // Using the package manager, this artifact can be obtained by the user
     // through `b.dependency(<name in build.zig.zon>, .{}).artifact("vulkan-zig-generator")`.
     // with that, the user need only `.addArg("path/to/vk.xml")`, and then obtain
     // a file source to the generated code with `.addOutputArg("vk.zig")`
     const generator_exe = b.addExecutable(.{
         .name = "vulkan-zig-generator",
-        .root_source_file = b.path("src/main.zig"),
+        .root_module = root_module,
-        .target = target,
-        .optimize = optimize,
     });
     b.installArtifact(generator_exe);
 
@@ -47,16 +51,16 @@ pub fn build(b: *std.Build) void {
         // It does not need to run anyway.
         const ref_all_decls_test = b.addObject(.{
             .name = "ref-all-decls-test",
+            .root_module = b.createModule(.{
                 .root_source_file = b.path("test/ref_all_decls.zig"),
                 .target = target,
                 .optimize = optimize,
+            }),
         });
         ref_all_decls_test.root_module.addImport("vulkan", vk_zig_module);
         test_step.dependOn(&ref_all_decls_test.step);
     }
 
-    const test_target = b.addTest(.{
+    const test_target = b.addTest(.{ .root_module = root_module });
-        .root_source_file = b.path("src/main.zig"),
-    });
     test_step.dependOn(&b.addRunArtifact(test_target).step);
 }

examples/build.zig

@@ -12,10 +12,14 @@ pub fn build(b: *std.Build) void {
 
     const triangle_exe = b.addExecutable(.{
         .name = "triangle",
+        .root_module = b.createModule(.{
             .root_source_file = b.path("triangle.zig"),
             .target = target,
             .link_libc = true,
             .optimize = optimize,
+        }),
+        // TODO: Remove this once x86_64 is stable
+        .use_llvm = true,
     });
     b.installArtifact(triangle_exe);
     triangle_exe.linkSystemLibrary("glfw");
@@ -33,17 +37,18 @@ pub fn build(b: *std.Build) void {
 
     if (use_zig_shaders) {
         const spirv_target = b.resolveTargetQuery(.{
-            .cpu_arch = .spirv64,
+            .cpu_arch = .spirv32,
             .os_tag = .vulkan,
             .cpu_model = .{ .explicit = &std.Target.spirv.cpu.vulkan_v1_2 },
-            .cpu_features_add = std.Target.spirv.featureSet(&.{.int64}),
             .ofmt = .spirv,
         });
 
         const vert_spv = b.addObject(.{
             .name = "vertex_shader",
+            .root_module = b.createModule(.{
                 .root_source_file = b.path("shaders/vertex.zig"),
                 .target = spirv_target,
+            }),
             .use_llvm = false,
         });
         triangle_exe.root_module.addAnonymousImport(
@@ -53,8 +58,10 @@ pub fn build(b: *std.Build) void {
 
         const frag_spv = b.addObject(.{
             .name = "fragment_shader",
+            .root_module = b.createModule(.{
                 .root_source_file = b.path("shaders/fragment.zig"),
                 .target = spirv_target,
+            }),
             .use_llvm = false,
         });
         triangle_exe.root_module.addAnonymousImport(

examples/shaders/fragment.zig

@@ -8,6 +8,5 @@ export fn main() callconv(.spirv_fragment) void {
     gpu.location(&v_color, 0);
     gpu.location(&f_color, 0);
 
-    const temp: @Vector(4, f32) = .{ v_color[0], v_color[1], v_color[2], 1.0 };
+    f_color = .{ v_color[0], v_color[1], v_color[2], 1.0 };
-    f_color = temp;
 }

src/id_render.zig

@@ -52,13 +52,8 @@ pub fn isZigPrimitiveType(name: []const u8) bool {
     return false;
 }
 
-pub fn writeIdentifier(writer: anytype, id: []const u8) !void {
+pub fn writeIdentifier(w: *std.io.Writer, id: []const u8) !void {
-    // https://github.com/ziglang/zig/issues/2897
+    try w.print("{f}", .{std.zig.fmtId(id)});
-    if (isZigPrimitiveType(id)) {
-        try writer.print("@\"{}\"", .{std.zig.fmtEscapes(id)});
-    } else {
-        try writer.print("{}", .{std.zig.fmtId(id)});
-    }
 }
 
 pub const CaseStyle = enum {
@@ -196,13 +191,13 @@ pub const IdRenderer = struct {
         }
     }
 
-    pub fn renderFmt(self: *IdRenderer, out: anytype, comptime fmt: []const u8, args: anytype) !void {
+    pub fn renderFmt(self: *IdRenderer, out: *std.Io.Writer, comptime fmt: []const u8, args: anytype) !void {
         self.text_cache.items.len = 0;
         try std.fmt.format(self.text_cache.writer(), fmt, args);
         try writeIdentifier(out, self.text_cache.items);
     }
 
-    pub fn renderWithCase(self: *IdRenderer, out: anytype, case_style: CaseStyle, id: []const u8) !void {
+    pub fn renderWithCase(self: *IdRenderer, out: *std.Io.Writer, case_style: CaseStyle, id: []const u8) !void {
         const tag = self.getAuthorTag(id);
         // The trailing underscore doesn't need to be removed here as its removed by the SegmentIterator.
         const adjusted_id = if (tag) |name| id[0 .. id.len - name.len] else id;

src/main.zig

@@ -8,17 +8,18 @@ fn invalidUsage(prog_name: []const u8, comptime fmt: []const u8, args: anytype)
 }
 
 fn reportParseErrors(tree: std.zig.Ast) !void {
-    const stderr = std.io.getStdErr().writer();
+    var buf: [1024]u8 = undefined;
-
+    var stderr = std.fs.File.stderr().writer(&buf);
+    const w = &stderr.interface;
     for (tree.errors) |err| {
         const loc = tree.tokenLocation(0, err.token);
-        try stderr.print("(vulkan-zig error):{}:{}: error: ", .{ loc.line + 1, loc.column + 1 });
+        try w.print("(vulkan-zig error):{}:{}: error: ", .{ loc.line + 1, loc.column + 1 });
-        try tree.renderError(err, stderr);
+        try tree.renderError(err, w);
-        try stderr.print("\n{s}\n", .{tree.source[loc.line_start..loc.line_end]});
+        try w.print("\n{s}\n", .{tree.source[loc.line_start..loc.line_end]});
         for (0..loc.column) |_| {
-            try stderr.writeAll(" ");
+            try w.writeAll(" ");
         }
-        try stderr.writeAll("^\n");
+        try w.writeAll("^\n");
     }
 }
 
@@ -41,7 +42,9 @@ pub fn main() !void {
     while (args.next()) |arg| {
         if (std.mem.eql(u8, arg, "--help") or std.mem.eql(u8, arg, "-h")) {
             @setEvalBranchQuota(2000);
-            std.io.getStdOut().writer().print(
+            var buf: [1024]u8 = undefined;
+            var w = std.fs.File.stdout().writer(&buf);
+            w.interface.print(
                 \\Utility to generate a Zig binding from the Vulkan XML API registry.
                 \\
                 \\The most recent Vulkan XML API registry can be obtained from
@@ -109,24 +112,12 @@ pub fn main() !void {
         null;
 
     var out_buffer = std.ArrayList(u8).init(allocator);
-    generator.generate(allocator, api, xml_src, maybe_video_xml_src, out_buffer.writer()) catch |err| switch (err) {
+    var w = out_buffer.writer().adaptToNewApi();
-        error.InvalidXml => {
+    generator.generate(allocator, api, xml_src, maybe_video_xml_src, &w.new_interface) catch {
-            std.log.err("invalid vulkan registry - invalid xml", .{});
+        std.process.fatal(
-            std.log.err("please check that the correct vk.xml file is passed", .{});
+            "generating failed. please check that the correct vk.xml file is passed",
-            std.process.exit(1);
+            .{},
-        },
+        );
-        error.InvalidRegistry => {
-            std.log.err("invalid vulkan registry - registry is valid xml but contents are invalid", .{});
-            std.log.err("please check that the correct vk.xml file is passed", .{});
-            std.process.exit(1);
-        },
-        error.UnhandledBitfieldStruct => {
-            std.log.err("unhandled struct with bit fields detected in vk.xml", .{});
-            std.log.err("this is a bug in vulkan-zig", .{});
-            std.log.err("please make a bug report at https://github.com/Snektron/vulkan-zig/issues/", .{});
-            std.process.exit(1);
-        },
-        error.OutOfMemory => @panic("oom"),
     };
 
     out_buffer.append(0) catch @panic("oom");

src/vulkan/generator.zig

@@ -186,7 +186,7 @@ pub const Generator = struct {
         self.registry.decls.len = i;
     }
 
-    fn render(self: *Generator, writer: anytype) !void {
+    fn render(self: *Generator, writer: *std.Io.Writer) !void {
         try renderRegistry(writer, self.arena.allocator(), &self.registry, &self.id_renderer, self.have_video);
     }
 };
@@ -204,7 +204,7 @@ pub fn generate(
     api: Api,
     spec_xml: []const u8,
     maybe_video_spec_xml: ?[]const u8,
-    writer: anytype,
+    writer: *std.Io.Writer,
 ) !void {
     const spec = xml.parse(allocator, spec_xml) catch |err| switch (err) {
         error.InvalidDocument,

src/vulkan/render.zig

@@ -17,15 +17,15 @@ const preamble =
     \\const Allocator = std.mem.Allocator;
     \\
     \\pub const vulkan_call_conv: std.builtin.CallingConvention = if (builtin.os.tag == .windows and builtin.cpu.arch == .x86)
-    \\        .Stdcall
+    \\        .winapi
     \\    else if (builtin.abi == .android and (builtin.cpu.arch.isARM() or builtin.cpu.arch.isThumb()) and std.Target.arm.featureSetHas(builtin.cpu.features, .has_v7) and builtin.cpu.arch.ptrBitWidth() == 32)
     \\        // On Android 32-bit ARM targets, Vulkan functions use the "hardfloat"
     \\        // calling convention, i.e. float parameters are passed in registers. This
     \\        // is true even if the rest of the application passes floats on the stack,
     \\        // as it does by default when compiling for the armeabi-v7a NDK ABI.
-    \\        .AAPCSVFP
+    \\        .arm_aapcs_vfp
     \\    else
-    \\        .C;
+    \\        .c;
     // Note: Keep in sync with flag_functions
     \\pub fn FlagsMixin(comptime FlagsType: type) type {
     \\    return struct {
@@ -336,11 +336,9 @@ pub fn trimVkNamespace(id: []const u8) []const u8 {
     return id;
 }
 
-fn Renderer(comptime WriterType: type) type {
+const Renderer = struct {
-    return struct {
     const Self = @This();
-        const WriteError = WriterType.Error;
+    const RenderTypeInfoError = std.Io.Writer.Error || std.fmt.ParseIntError || error{ OutOfMemory, InvalidRegistry };
-        const RenderTypeInfoError = WriteError || std.fmt.ParseIntError || error{ OutOfMemory, InvalidRegistry };
 
     const BitflagName = struct {
         /// Name without FlagBits, so VkSurfaceTransformFlagBitsKHR
@@ -374,7 +372,7 @@ fn Renderer(comptime WriterType: type) type {
         },
     };
 
-        writer: WriterType,
+    writer: *std.Io.Writer,
     allocator: Allocator,
     registry: *const reg.Registry,
     id_renderer: *IdRenderer,
@@ -383,7 +381,7 @@ fn Renderer(comptime WriterType: type) type {
     have_video: bool,
 
     fn init(
-            writer: WriterType,
+        writer: *std.Io.Writer,
         allocator: Allocator,
         registry: *const reg.Registry,
         id_renderer: *IdRenderer,
@@ -2114,17 +2112,16 @@ fn Renderer(comptime WriterType: type) type {
             try self.writeIdentifierWithCase(.title, trimVkNamespace(name));
         }
     }
-    };
+};
-}
 
 pub fn render(
-    writer: anytype,
+    writer: *std.Io.Writer,
     allocator: Allocator,
     registry: *const reg.Registry,
     id_renderer: *IdRenderer,
     have_video: bool,
 ) !void {
-    var renderer = try Renderer(@TypeOf(writer)).init(writer, allocator, registry, id_renderer, have_video);
+    var renderer = try Renderer.init(writer, allocator, registry, id_renderer, have_video);
     defer renderer.deinit();
     try renderer.render();
 }

test/ref_all_decls.zig

@@ -21,80 +21,76 @@ pub const MTLTexture_id = u32;
 pub const MTLSharedEvent_id = u32;
 pub const IOSurfaceRef = u32;
 
-// For some reason these types are exported in a different header, and not described in vk.xml.
+pub const StdVideoH264ProfileIdc = u32;
-// If we are not also generating these, the user will have to manually specify them.
+pub const StdVideoH264LevelIdc = u32;
-pub usingnamespace if (!vk.have_vulkan_video) struct {
+pub const StdVideoH264ChromaFormatIdc = u32;
-    pub const StdVideoH264ProfileIdc = u32;
+pub const StdVideoH264PocType = u32;
-    pub const StdVideoH264LevelIdc = u32;
+pub const StdVideoH264SpsFlags = u32;
-    pub const StdVideoH264ChromaFormatIdc = u32;
+pub const StdVideoH264ScalingLists = u32;
-    pub const StdVideoH264PocType = u32;
+pub const StdVideoH264SequenceParameterSetVui = u32;
-    pub const StdVideoH264SpsFlags = u32;
+pub const StdVideoH264AspectRatioIdc = u32;
-    pub const StdVideoH264ScalingLists = u32;
+pub const StdVideoH264HrdParameters = u32;
-    pub const StdVideoH264SequenceParameterSetVui = u32;
+pub const StdVideoH264SpsVuiFlags = u32;
-    pub const StdVideoH264AspectRatioIdc = u32;
+pub const StdVideoH264WeightedBipredIdc = u32;
-    pub const StdVideoH264HrdParameters = u32;
+pub const StdVideoH264PpsFlags = u32;
-    pub const StdVideoH264SpsVuiFlags = u32;
+pub const StdVideoH264SliceType = u32;
-    pub const StdVideoH264WeightedBipredIdc = u32;
+pub const StdVideoH264CabacInitIdc = u32;
-    pub const StdVideoH264PpsFlags = u32;
+pub const StdVideoH264DisableDeblockingFilterIdc = u32;
-    pub const StdVideoH264SliceType = u32;
+pub const StdVideoH264PictureType = u32;
-    pub const StdVideoH264CabacInitIdc = u32;
+pub const StdVideoH264ModificationOfPicNumsIdc = u32;
-    pub const StdVideoH264DisableDeblockingFilterIdc = u32;
+pub const StdVideoH264MemMgmtControlOp = u32;
-    pub const StdVideoH264PictureType = u32;
+pub const StdVideoDecodeH264PictureInfo = u32;
-    pub const StdVideoH264ModificationOfPicNumsIdc = u32;
+pub const StdVideoDecodeH264ReferenceInfo = u32;
-    pub const StdVideoH264MemMgmtControlOp = u32;
+pub const StdVideoDecodeH264PictureInfoFlags = u32;
-    pub const StdVideoDecodeH264PictureInfo = u32;
+pub const StdVideoDecodeH264ReferenceInfoFlags = u32;
-    pub const StdVideoDecodeH264ReferenceInfo = u32;
+pub const StdVideoH264SequenceParameterSet = u32;
-    pub const StdVideoDecodeH264PictureInfoFlags = u32;
+pub const StdVideoH264PictureParameterSet = u32;
-    pub const StdVideoDecodeH264ReferenceInfoFlags = u32;
+pub const StdVideoH265ProfileIdc = u32;
-    pub const StdVideoH264SequenceParameterSet = u32;
+pub const StdVideoH265VideoParameterSet = u32;
-    pub const StdVideoH264PictureParameterSet = u32;
+pub const StdVideoH265SequenceParameterSet = u32;
-    pub const StdVideoH265ProfileIdc = u32;
+pub const StdVideoH265PictureParameterSet = u32;
-    pub const StdVideoH265VideoParameterSet = u32;
+pub const StdVideoH265DecPicBufMgr = u32;
-    pub const StdVideoH265SequenceParameterSet = u32;
+pub const StdVideoH265HrdParameters = u32;
-    pub const StdVideoH265PictureParameterSet = u32;
+pub const StdVideoH265VpsFlags = u32;
-    pub const StdVideoH265DecPicBufMgr = u32;
+pub const StdVideoH265LevelIdc = u32;
-    pub const StdVideoH265HrdParameters = u32;
+pub const StdVideoH265SpsFlags = u32;
-    pub const StdVideoH265VpsFlags = u32;
+pub const StdVideoH265ScalingLists = u32;
-    pub const StdVideoH265LevelIdc = u32;
+pub const StdVideoH265SequenceParameterSetVui = u32;
-    pub const StdVideoH265SpsFlags = u32;
+pub const StdVideoH265PredictorPaletteEntries = u32;
-    pub const StdVideoH265ScalingLists = u32;
+pub const StdVideoH265PpsFlags = u32;
-    pub const StdVideoH265SequenceParameterSetVui = u32;
+pub const StdVideoH265SubLayerHrdParameters = u32;
-    pub const StdVideoH265PredictorPaletteEntries = u32;
+pub const StdVideoH265HrdFlags = u32;
-    pub const StdVideoH265PpsFlags = u32;
+pub const StdVideoH265SpsVuiFlags = u32;
-    pub const StdVideoH265SubLayerHrdParameters = u32;
+pub const StdVideoH265SliceType = u32;
-    pub const StdVideoH265HrdFlags = u32;
+pub const StdVideoH265PictureType = u32;
-    pub const StdVideoH265SpsVuiFlags = u32;
+pub const StdVideoDecodeH265PictureInfo = u32;
-    pub const StdVideoH265SliceType = u32;
+pub const StdVideoDecodeH265ReferenceInfo = u32;
-    pub const StdVideoH265PictureType = u32;
+pub const StdVideoDecodeH265PictureInfoFlags = u32;
-    pub const StdVideoDecodeH265PictureInfo = u32;
+pub const StdVideoDecodeH265ReferenceInfoFlags = u32;
-    pub const StdVideoDecodeH265ReferenceInfo = u32;
+pub const StdVideoAV1Profile = u32;
-    pub const StdVideoDecodeH265PictureInfoFlags = u32;
+pub const StdVideoAV1Level = u32;
-    pub const StdVideoDecodeH265ReferenceInfoFlags = u32;
+pub const StdVideoAV1SequenceHeader = u32;
-    pub const StdVideoAV1Profile = u32;
+pub const StdVideoDecodeAV1PictureInfo = u32;
-    pub const StdVideoAV1Level = u32;
+pub const StdVideoDecodeAV1ReferenceInfo = u32;
-    pub const StdVideoAV1SequenceHeader = u32;
+pub const StdVideoEncodeH264SliceHeader = u32;
-    pub const StdVideoDecodeAV1PictureInfo = u32;
+pub const StdVideoEncodeH264PictureInfo = u32;
-    pub const StdVideoDecodeAV1ReferenceInfo = u32;
+pub const StdVideoEncodeH264ReferenceInfo = u32;
-    pub const StdVideoEncodeH264SliceHeader = u32;
+pub const StdVideoEncodeH264SliceHeaderFlags = u32;
-    pub const StdVideoEncodeH264PictureInfo = u32;
+pub const StdVideoEncodeH264ReferenceListsInfo = u32;
-    pub const StdVideoEncodeH264ReferenceInfo = u32;
+pub const StdVideoEncodeH264PictureInfoFlags = u32;
-    pub const StdVideoEncodeH264SliceHeaderFlags = u32;
+pub const StdVideoEncodeH264ReferenceInfoFlags = u32;
-    pub const StdVideoEncodeH264ReferenceListsInfo = u32;
+pub const StdVideoEncodeH264RefMgmtFlags = u32;
-    pub const StdVideoEncodeH264PictureInfoFlags = u32;
+pub const StdVideoEncodeH264RefListModEntry = u32;
-    pub const StdVideoEncodeH264ReferenceInfoFlags = u32;
+pub const StdVideoEncodeH264RefPicMarkingEntry = u32;
-    pub const StdVideoEncodeH264RefMgmtFlags = u32;
+pub const StdVideoEncodeH265PictureInfoFlags = u32;
-    pub const StdVideoEncodeH264RefListModEntry = u32;
+pub const StdVideoEncodeH265PictureInfo = u32;
-    pub const StdVideoEncodeH264RefPicMarkingEntry = u32;
+pub const StdVideoEncodeH265SliceSegmentHeader = u32;
-    pub const StdVideoEncodeH265PictureInfoFlags = u32;
+pub const StdVideoEncodeH265ReferenceInfo = u32;
-    pub const StdVideoEncodeH265PictureInfo = u32;
+pub const StdVideoEncodeH265ReferenceListsInfo = u32;
-    pub const StdVideoEncodeH265SliceSegmentHeader = u32;
+pub const StdVideoEncodeH265SliceSegmentHeaderFlags = u32;
-    pub const StdVideoEncodeH265ReferenceInfo = u32;
+pub const StdVideoEncodeH265ReferenceInfoFlags = u32;
-    pub const StdVideoEncodeH265ReferenceListsInfo = u32;
+pub const StdVideoEncodeH265ReferenceModificationFlags = u32;
-    pub const StdVideoEncodeH265SliceSegmentHeaderFlags = u32;
+pub const StdVideoEncodeAV1OperatingPointInfo = u32;
-    pub const StdVideoEncodeH265ReferenceInfoFlags = u32;
-    pub const StdVideoEncodeH265ReferenceModificationFlags = u32;
-    pub const StdVideoEncodeAV1OperatingPointInfo = u32;
-} else struct {};
 
 comptime {
     @setEvalBranchQuota(1000000);