Skip to content

[bazel] add new cmakedefine from #147418 to bazel config file #150912

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

Merged
merged 1 commit into from
Jul 28, 2025

Conversation

ingomueller-net
Copy link
Contributor

This PR adds the #cmakedefine LLVM_ENABLE_PROFCHECK in llvm-config.h.cmake introduced in #147418 to the copy of that file in the bazel overlay directory such that that define is also avalable in the bazel build. Not having the define broke the bazel build.

This PR adds the `#cmakedefine LLVM_ENABLE_PROFCHECK` in
`llvm-config.h.cmake` introduced in llvm#147418 to the copy of that file in
the bazel overlay directory such that that define is also avalable in
the bazel build. Not having the define broke the bazel build.

Signed-off-by: Ingo Müller <[email protected]>
@llvmbot llvmbot added the bazel "Peripheral" support tier build system: utils/bazel label Jul 28, 2025
Copy link
Member

@akuegel akuegel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@akuegel akuegel merged commit 5ad7ef6 into llvm:main Jul 28, 2025
11 checks passed
@ingomueller-net ingomueller-net deleted the fix-bazel-profcheck-define branch July 28, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bazel "Peripheral" support tier build system: utils/bazel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants