-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
compiler segfault #16874
Comments
Here's the output I get with a debug build of Zig
zig test command
zig build-exe commandFrom what I can tell from the ZIR referenced in these outputs, both may be related to the issue reported in #16412, namely attempting to coerce
|
Bisecting seems to point to PR #15726 merged in commit 496320d. More specifically, that commit cannot run the allocator,
bench_name,
test_fn.func,
- .{},
+ std.meta.ArgsTuple(fn () void){},
options,
max_samples,
&progress, makes the |
FYI, this is working for me as of 0.12.0-dev.722+412d863ba (see also #17380 which adds a behavior test for this case). |
Zig Version
0.11.0 and 0.12.0-dev.125+7ef1eb1c2
Steps to Reproduce and Observed Behavior
Clone https://github.com/dweiller/zubench (commit 83cbcff) and run
zig build run
. More minimally, run either ofzig test examples/fib.zig --test-runner src/bench_runner.zig --mod zubench::src/bench.zig --deps zubench zig build-exe src/bench_runner.zig --mod @bench:zubench:examples/fib_build.zig --mod zubench::src/bench.zig --deps @bench,zubench
Expected Behavior
With
zig build run
there should be no error, with thezig test
line above there should be a failing test (the failing one is removed with--test-filter fib
); thebuild-exe
should compile without issue.I intend to update this issue after attempting to bisect the zig compiler to work out exactly when this segfault was introducedI have bisected the issue - details below.The text was updated successfully, but these errors were encountered: