Skip to content
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.

Sign up for GitHub

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

Jump to bottom

improve support for clang #16

Open
kazuho opened this issue May 9, 2015 · 0 comments
Open

improve support for clang #16

kazuho opened this issue May 9, 2015 · 0 comments

Comments

@kazuho
Copy link
Member

kazuho commented May 9, 2015

As of now, we have a code that emulates the behavior of gcc -wrapper in case clang (or other compiler) is being used.

However the argument parsing logic of the emulation is horrible; it just strips -o target and assumes that the last argument is the name of the source file. It is evident that such approach will fail in certain cases, e.g. when -Wl, is used after specifying the source file, or, when more than one source files are used.

We need to implement a better option parser like the one found in ccache or dist-cc.
kazuho added a commit that referenced this issue May 11, 2015
@kazuho
[clang] extract source files from argument to clang using the followi…
8923fea 
…ng rules:

* does not start with `-`
* is not a succeeding argument of `-o`
* the file exists
* the suffix is either of: .c, .m, .mm, .M, .cc, .cp, .cxx, .cpp, .CPP, .c++, .C

relates to #16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kazuho