plucky turtle
So I updated master and I get this error. Maybe it's a cache error but I deleted zig-cache and ~/.cache/zig...
Zig version: 0.11.0-dev.1430+ce6de2df8
Possible problematic commit: https://github.com/ziglang/zig/commit/7ea3937c5a6f63557db99c8a8f7b3f6d3de8c38b
velvet basin
I just gave you the reasoning why it failed
The CLI expects -mno-red-zone but the build step passes -fno-red-zone
plucky turtle
I just gave you the reasoning why it failed
but searching through the repo I didn't find any reference to -fno-red-zone
velvet basin
It's literally in the code you posted here. Look at the function addFlag, it will prepend the flag with -f. But that's incorrect for red-zone. The CLI should however be updated to also accept -f to make it consistant with other flags.
plucky turtle
oh I see, thanks. I should file an issue, right?
velvet basin
Or better, a PR to fix it 😛
plucky turtle
ok so I do append -mno-red-zone right?
velvet basin
The PR should update the CLI here: https://github.com/ziglang/zig/blob/master/src/main.zig#L1180-L1182 to accept -fno-red-zone instead of -mno-red-zone and also update the usage at https://github.com/ziglang/zig/blob/master/src/main.zig#L395-L396
plucky turtle
The PR should update the CLI here: <https://github.com/ziglang/zig/blob/master/s...
one second, this should follow GCC/Clang flag style? because the mno-red-zone is the one being used by them
velvet basin
In that case make the build step pass -mno-red-zone instead