zig cc can't accept -- in args

[iacore]

Zig Version

0.11.0-dev.1898+36d47dd19

Steps to Reproduce and Observed Behavior

> echo "int main() {}" > program.c

> zig cc -- program.c
error: Unknown Clang option: '--'

> clang -- program.c
# success

Expected Behavior

It should parse like clang. args after -- are not flags.

[ghost]

Are you sure that's actually how clang works? I ran clang -- -program.c where -program.c exists and it said:

error: unknown argument: '-program.c'

[iacore]

-program.c should be a file name, if your C file starts with -.

[ghost]

Yes, clang for me shows that error message when -program.c is the name of a C source file. My clang version is 15.0.7, what version are you on?

[iacore]

Yes, clang for me shows that error message when -program.c is the name of a C source file. My clang version is 15.0.7, what version are you on?

I gave a bad example. What about clang -- program.c

[ghost]

Strange, I've found that actually clang for me does process the -- as expected, even with -program.c... but only if -program.c doesn't exist.

clang -- -program.c where -program.c doesn't exist:

clang-15: error: no such file or directory: '-program.c'
clang-15: error: no input files

clang -- -program.c where it does exist:

error: unknown argument: '-program.c'

[iacore]

Did we find a bug in clang?

[ghost]

I dunno, seems like it 🤷

[andrewrk]

This is not demonstrated to be a bug yet. Please improve the bug report

[iacore]

This is not demonstrated to be a bug yet. Please improve the bug report

Updated

[hadeutscher]

Hey,

Why doesn't zigcc accept the -- delimiter?

While this isn't technically a bug, it does cause quite a lot of trouble for the rest of the community - for example, the Rust cc crate has special-case handling for the zig compiler which removes the -- delimiter, which breaks every once in a while, such as yesterday, and wreaks havoc on projects built using the dist framework (which uses zigcc internally for cross-compiling aarch64 targets).

Also, clang accepts this delimiter, so if zigcc aims to be a drop-in replacement for clang, it really is a bug.

Seeing as you already wrote the pull request that fixes this, maybe we can just get that one merged and be over with this?

Thanks