Skip to content

[libc] Fix bazel build #81980

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

Closed
wants to merge 2 commits into from
Closed

[libc] Fix bazel build #81980

wants to merge 2 commits into from

Conversation

gchatelet
Copy link
Contributor

@gchatelet gchatelet commented Feb 16, 2024

Broken by #81819

@gchatelet gchatelet requested a review from lntue February 16, 2024 10:52
@llvmbot llvmbot added the libc label Feb 16, 2024
@llvmbot
Copy link
Member

llvmbot commented Feb 16, 2024

@llvm/pr-subscribers-libc

Author: Guillaume Chatelet (gchatelet)

Changes

Broken by #81819


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

2 Files Affected:

  • (modified) utils/bazel/llvm-project-overlay/libc/BUILD.bazel (+5)
  • (modified) utils/bazel/llvm-project-overlay/libc/test/UnitTest/BUILD.bazel (+2)
diff --git a/utils/bazel/llvm-project-overlay/libc/BUILD.bazel b/utils/bazel/llvm-project-overlay/libc/BUILD.bazel
index 1518d7910299d0..1998c95dcbe9c5 100644
--- a/utils/bazel/llvm-project-overlay/libc/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/libc/BUILD.bazel
@@ -891,6 +891,11 @@ libc_support_library(
     ],
 )
 
+libc_support_library(
+    name = "__support_fixed_point_fx_rep",
+    hdrs = ["src/__support/fixed_point/fx_rep.h"],
+)
+
 libc_support_library(
     name = "__support_osutil_syscall",
     hdrs = ["src/__support/OSUtil/syscall.h"],
diff --git a/utils/bazel/llvm-project-overlay/libc/test/UnitTest/BUILD.bazel b/utils/bazel/llvm-project-overlay/libc/test/UnitTest/BUILD.bazel
index cffaa27d1df18e..fd4bf4a4db74cc 100644
--- a/utils/bazel/llvm-project-overlay/libc/test/UnitTest/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/libc/test/UnitTest/BUILD.bazel
@@ -46,6 +46,7 @@ libc_support_library(
         "//libc:__support_cpp_string",
         "//libc:__support_cpp_string_view",
         "//libc:__support_cpp_type_traits",
+        "//libc:__support_fixed_point_fx_rep",
         "//libc:__support_fputil_fp_bits",
         "//libc:__support_fputil_fpbits_str",
         "//libc:__support_fputil_rounding_mode",
@@ -53,6 +54,7 @@ libc_support_library(
         "//libc:__support_stringutil",
         "//libc:__support_uint128",
         "//libc:errno",
+        "//libc:llvm_libc_macros_stdfix_macros",
         "//llvm:Support",
     ],
 )

@gchatelet
Copy link
Contributor Author

This has been fixed in the meantime. Dropping this patch.

@gchatelet gchatelet closed this Feb 16, 2024
@gchatelet gchatelet deleted the fix_bazel branch February 16, 2024 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants