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.

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

Can't utilize ccache with a custom compiler #253

Closed
vlovich opened this issue Aug 18, 2021 · 0 comments · Fixed by #255
Closed

Can't utilize ccache with a custom compiler #253

vlovich opened this issue Aug 18, 2021 · 0 comments · Fixed by #255

Comments

@vlovich
Copy link
Contributor

vlovich commented Aug 18, 2021

Typically one might expect to be able to supply make WASM_CC="ccache <path to clang>" to override the compiler & use ccache, but the use of double-quotes invoking the compiler ("$(WASM_CC)") inhibits this. The double-quotes should be removed or perhaps a new WASM_CC_WRAPPER introduced that can be used to inject ccache?

vlovich added a commit to vlovich/wasi-libc that referenced this issue Sep 1, 2021
If using a custom toolchain not on the path, explicitly invoking
ccache/sccache is required (also easier to debug that the right compiler
is being invoked). Fixes WebAssembly#253.
vlovich added a commit to vlovich/wasi-libc that referenced this issue Sep 8, 2021
If using a custom toolchain not on the path, explicitly invoking
ccache/sccache is required (also easier to debug that the right compiler
is being invoked). Fixes WebAssembly#253.
sbc100 pushed a commit that referenced this issue Sep 22, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
If using a custom toolchain not on the path, explicitly invoking
ccache/sccache is required (also easier to debug that the right compiler
is being invoked). Fixes #253.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant