shylie/llvm-project
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[bazel] Enable header processing for C++ builds. (#138934)

Browse Source 
This would only work for projects that explicitly enable "parse_headers"
feature (or for builds that specify this on command-line) - right now
there are none, so this change shouldn't affect most builds. When
"parse_headers" is enabled though, it would catch problems of
incorrect/missing includes in header-only cc_libraries.

See https://bazel.build/docs/bazel-and-cpp#toolchain-features on why
this option is a best practice for C++ projects.
This commit is contained in:
Alexey Samsonov 2025-05-07 22:18:05 -07:00 committed by GitHub
parent 28521368d7
commit 215dbcb2bc
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
utils/bazel/.bazelrc
View File

@ -58,6 +58,11 @@ build --experimental_cc_shared_library
# https://github.com/bazelbuild/bazel/commit/03246077f948f2790a83520e7dccc2625650e6df
build --build_runfile_links=false
# Enable header processing to verify that header-only libraries are
# self-contained (for sub-projects that enable "parse_headers" feature).
# See https://bazel.build/docs/bazel-and-cpp#toolchain-features
build --process_headers_in_dependencies
###############################################################################
# Options to select different strategies for linking potential dependent
# libraries. The default leaves it disabled.