zig 0.15.1

[dasimmet]

No description provided.

[malcolmstill]

Thanks for pulling this together @dasimmet! Looking good so far, but I have a question on the .always_tail change (which I will add to the particular line(s)).

[dasimmet]

Ah yeah thanks for clarifying, I suppose we can also switch on the compiler backend instead of build mode, so debug builds with llvm can still use tail calls.
The fib example already segfaults in debug x86_64 i noticed.

The remaining errors in the test seems like a bug in zig's json parser in the 0.15.1 release...

[malcolmstill]

Ah yeah thanks for clarifying, I suppose we can also switch on the compiler backend instead of build mode, so debug builds with llvm can still use tail calls. The fib example already segfaults in debug x86_64 i noticed.

The remaining errors in the test seems like a bug in zig's json parser in the 0.15.1 release...

I've fixed the json parser issue by reducing the width of the integer for the line field (see 189017d).

The testsuite still fails of course (but only on a few of the tests) because that will default to a debug build. Again we have options:

1. Run the testsuite in CI with a --release build to get llvm
2. Insist on llvm even in debug, e.g.

index e1c8e46..974992d 100644
--- i/build.zig
+++ w/build.zig
@@ -34,14 +34,15 @@ pub fn build(b: *Build) !void {
     const testrunner = b.addExecutable(.{
         .name = "testrunner",
         .root_module = b.addModule("testrunner", .{
             .root_source_file = b.path("test/testrunner/src/testrunner.zig"),
             .target = target,
             .optimize = optimize,
         }),
+        .use_llvm = true,
     });
     testrunner.root_module.addImport("zware", zware_module);
 
     const testsuite_dep = b.dependency("testsuite", .{});
 
     const testsuite_step = b.step("testsuite", "Run all the testsuite tests");
     for (test_names) |test_name| {

[malcolmstill]

Thanks @dasimmet and @marler8997!