[llvm] [libc][bazel] fix missing iterator inclusion (PR #85840)

via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 19 11:29:19 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libc

Author: Michael Jones (michaelrj-google)

<details>
<summary>Changes</summary>

Patch #<!-- -->85702 added a new iterator header for array to use, but didn't
add it as a dependency in the bazel build. This patch fixes the issue.


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


1 Files Affected:

- (modified) utils/bazel/llvm-project-overlay/libc/BUILD.bazel (+10) 


``````````diff
diff --git a/utils/bazel/llvm-project-overlay/libc/BUILD.bazel b/utils/bazel/llvm-project-overlay/libc/BUILD.bazel
index 77fb8c16507ba1..4766e168daadbd 100644
--- a/utils/bazel/llvm-project-overlay/libc/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/libc/BUILD.bazel
@@ -161,10 +161,20 @@ libc_support_library(
     ],
 )
 
+libc_support_library(
+    name = "__support_cpp_iterator",
+    hdrs = ["src/__support/CPP/iterator.h"],
+    deps = [
+        ":__support_cpp_type_traits",
+        ":__support_macros_attributes",
+    ],
+)
+
 libc_support_library(
     name = "__support_cpp_array",
     hdrs = ["src/__support/CPP/array.h"],
     deps = [
+        ":__support_cpp_iterator",
         ":__support_macros_attributes",
     ],
 )

``````````

</details>


https://github.com/llvm/llvm-project/pull/85840


More information about the llvm-commits mailing list