Skip to content

Conversation

@ingomueller-net
Copy link
Contributor

This adds two dependencies that I wanted to be part of #151090 but forgot to add them before the last push, which related to wcschr.cpp and were originally broken by #150661.

This adds two dependencies that I wanted to be part of llvm#151090 but
forgot to add them before the last push, which related to
`wcschr.cpp` and were originally broken by llvm#150661.

Signed-off-by: Ingo Müller <[email protected]>
@ingomueller-net ingomueller-net requested a review from lntue July 29, 2025 07:06
@llvmbot llvmbot added libc bazel "Peripheral" support tier build system: utils/bazel labels Jul 29, 2025
@llvmbot
Copy link
Member

llvmbot commented Jul 29, 2025

@llvm/pr-subscribers-libc

Author: Ingo Müller (ingomueller-net)

Changes

This adds two dependencies that I wanted to be part of #151090 but forgot to add them before the last push, which related to wcschr.cpp and were originally broken by #150661.


Full diff: https://github.com/llvm/llvm-project/pull/151093.diff

1 Files Affected:

  • (modified) utils/bazel/llvm-project-overlay/libc/BUILD.bazel (+2)
diff --git a/utils/bazel/llvm-project-overlay/libc/BUILD.bazel b/utils/bazel/llvm-project-overlay/libc/BUILD.bazel
index d9e0110c437a6..e06110a75e771 100644
--- a/utils/bazel/llvm-project-overlay/libc/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/libc/BUILD.bazel
@@ -6252,7 +6252,9 @@ libc_function(
     deps = [
         ":__support_common",
         ":__support_macros_config",
+        ":__support_macros_null_check",
         ":types_wchar_t",
+        ":wchar_utils",
     ],
 )
 

@ingomueller-net ingomueller-net merged commit 4745637 into llvm:main Jul 29, 2025
12 checks passed
@ingomueller-net ingomueller-net deleted the fix-wcspbrk-bazel-deps branch July 29, 2025 07:30
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 libc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants