-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Force use of Clang over C++ modules with use_module_maps
#22660
Conversation
With `-std=c++20`, Clang defaults to using C++ modules rather than Clang modules, which can result in failures when using `layering_check` and thus `use_module_maps`.
@bazel-io flag |
@oquenchil The flag unfortunately requires clang-14: llvm/llvm-project@0556138 I will add version detection logic. |
@bazel-io fork 7.3.0 |
I added feature detection. |
With `-std=c++20`, Clang defaults to using C++ modules rather than Clang modules, which can result in failures when using `layering_check` and thus `use_module_maps`. Fixes bazel-contrib/toolchains_llvm#334 Closes bazelbuild#22660. PiperOrigin-RevId: 642548168 Change-Id: Ie1dd4c482d716635f7b6c53ffd3c5d84b6aa8cc2
…22720) With `-std=c++20`, Clang defaults to using C++ modules rather than Clang modules, which can result in failures when using `layering_check` and thus `use_module_maps`. Fixes bazel-contrib/toolchains_llvm#334 Closes #22660. PiperOrigin-RevId: 642548168 Change-Id: Ie1dd4c482d716635f7b6c53ffd3c5d84b6aa8cc2 Commit 9e913fb Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
The changes in this PR have been included in Bazel 7.3.0 RC1. Please test out the release candidate and report any issues as soon as possible. |
With
-std=c++20
, Clang defaults to using C++ modules rather than Clang modules, which can result in failures when usinglayering_check
and thususe_module_maps
.Fixes bazel-contrib/toolchains_llvm#334