Disable /Zi when using ccache on Windows/MSVC.
#11841
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ScottTodd
added
infrastructure
ScottTodd
commented
Jan 14, 2023
Comment on lines
+436
to
+441
| message(STATUS "Replacing /Zi with /Z7 since ccache is in use and does not support /Zi") | ||
| # https://learn.microsoft.com/en-us/cpp/build/reference/z7-zi-zi-debug-information-format | ||
| string(REPLACE "/Zi" "/Z7" CMAKE_C_FLAGS_DEBUG "${CMAKE_C_FLAGS_DEBUG}") | ||
| string(REPLACE "/Zi" "/Z7" CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG}") | ||
| string(REPLACE "/Zi" "/Z7" CMAKE_C_FLAGS_RELWITHDEBINFO "${CMAKE_C_FLAGS_RELWITHDEBINFO}") | ||
| string(REPLACE "/Zi" "/Z7" CMAKE_CXX_FLAGS_RELWITHDEBINFO "${CMAKE_CXX_FLAGS_RELWITHDEBINFO}") |
There was a problem hiding this comment.
Note: this will affect downstream users too. Not really a fan of when projects modify global flags, but I think this is niche enough...
There was a problem hiding this comment.
since it's conditional on ccache usage and anyone using ccache would need the same thing it seems ok - may still conflict with a user's similar hacks but probably worth waiting for that to be an issue :)
jpienaar
pushed a commit
that referenced
this pull request
May 1, 2023
One step towards enabling [ccache](https://ccache.dev/) on our Windows CI, but there are a few details to still work through: #11009 (comment). On my machine, I see these results (sample size 1): > clean build (no cache): 528 seconds > ``` > λ ccache --show-stats > Cacheable calls: 3942 / 3943 (99.97%) > Hits: 2 / 3942 ( 0.05%) > Direct: 0 / 2 ( 0.00%) > Preprocessed: 2 / 2 (100.0%) > Misses: 3940 / 3942 (99.95%) > Uncacheable calls: 1 / 3943 ( 0.03%) > Local storage: > Cache size (GB): 2.21 / 5.00 (44.21%) > Cleanups: 16 > ``` > clean build (with cache): 96 seconds > ``` > λ ccache --show-stats > Cacheable calls: 3942 / 3943 (99.97%) > Hits: 3939 / 3942 (99.92%) > Direct: 3939 / 3939 (100.0%) > Preprocessed: 0 / 3939 ( 0.00%) > Misses: 3 / 3942 ( 0.08%) > Uncacheable calls: 1 / 3943 ( 0.03%) > Local storage: > Cache size (GB): 2.21 / 5.00 (44.23%) > ``` My only changes to enable ccache were: * Download ccache.exe and put it on my `PATH` * Configure CMake with `-DCMAKE_C_COMPILER_LAUNCHER=ccache -DCMAKE_CXX_COMPILER_LAUNCHER=ccache` (added to `"cmake.configureArgs": [ ... ]` in VSCode settings)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
One step towards enabling ccache on our Windows CI, but there are a few details to still work through: #11009 (comment).
On my machine, I see these results (sample size 1):
My only changes to enable ccache were:
PATH-DCMAKE_C_COMPILER_LAUNCHER=ccache -DCMAKE_CXX_COMPILER_LAUNCHER=ccache(added to"cmake.configureArgs": [ ... ]in VSCode settings)