[libc-commits] [PATCH] D125490: [libc] fix uint includes and libc bazel

Michael Jones via Phabricator via libc-commits libc-commits at lists.llvm.org
Thu May 12 11:40:30 PDT 2022


michaelrj created this revision.
Herald added subscribers: libc-commits, ecnelises, tschuett.
Herald added projects: libc-project, All.
michaelrj requested review of this revision.

This patch fixes the includes for the new UInt class so that the api
test now passes, additionally it fixes the bazel files to account for
the new dependencies.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D125490

Files:
  libc/src/__support/CPP/UInt.h
  utils/bazel/llvm-project-overlay/libc/BUILD.bazel


Index: utils/bazel/llvm-project-overlay/libc/BUILD.bazel
===================================================================
--- utils/bazel/llvm-project-overlay/libc/BUILD.bazel
+++ utils/bazel/llvm-project-overlay/libc/BUILD.bazel
@@ -80,12 +80,20 @@
     deps = [":libc_root"],
 )
 
+cc_library(
+    name = "__support_cpp_uint",
+    hdrs = [
+        "src/__support/CPP/UInt.h",
+    ],
+    deps = [":libc_root","__support_cpp_array"],
+)
+
 cc_library(
     name = "__support_cpp_type_traits",
     hdrs = [
         "src/__support/CPP/TypeTraits.h",
     ],
-    deps = [":libc_root"],
+    deps = [":libc_root","__support_cpp_uint"],
 )
 
 cc_library(
Index: libc/src/__support/CPP/UInt.h
===================================================================
--- libc/src/__support/CPP/UInt.h
+++ libc/src/__support/CPP/UInt.h
@@ -9,7 +9,7 @@
 #ifndef LLVM_LIBC_UTILS_CPP_UINT_H
 #define LLVM_LIBC_UTILS_CPP_UINT_H
 
-#include "src/__support/CPP/Array.h"
+#include "Array.h"
 
 #include <stddef.h> // For size_t
 #include <stdint.h>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D125490.429035.patch
Type: text/x-patch
Size: 1040 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libc-commits/attachments/20220512/c3a431e2/attachment.bin>


More information about the libc-commits mailing list