Skip to content

[libc][complex] fix buildbot errors #121141

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
Dec 26, 2024
Merged

Conversation

Sh0g0-1758
Copy link
Member

Fix buildbot errors due to #121140

@llvmbot llvmbot added the libc label Dec 26, 2024
@llvmbot
Copy link
Member

llvmbot commented Dec 26, 2024

@llvm/pr-subscribers-libc

Author: Shourya Goel (Sh0g0-1758)

Changes

Fix buildbot errors due to #121140


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

2 Files Affected:

  • (modified) libc/config/linux/aarch64/entrypoints.txt (+2-1)
  • (modified) libc/config/linux/riscv/entrypoints.txt (+1)
diff --git a/libc/config/linux/aarch64/entrypoints.txt b/libc/config/linux/aarch64/entrypoints.txt
index b096b95b9472e3..00f0c6a8bfb8e4 100644
--- a/libc/config/linux/aarch64/entrypoints.txt
+++ b/libc/config/linux/aarch64/entrypoints.txt
@@ -629,7 +629,8 @@ if(LIBC_TYPES_HAS_CFLOAT16)
   )
 endif()
 
-if(LIBC_TYPES_HAS_FLOAT16)    
+if(LIBC_TYPES_HAS_FLOAT16)
+  list(APPEND TARGET_LIBM_ENTRYPOINTS
     # math.h C23 _Float16 entrypoints
     libc.src.math.canonicalizef16
     libc.src.math.ceilf16
diff --git a/libc/config/linux/riscv/entrypoints.txt b/libc/config/linux/riscv/entrypoints.txt
index 643e20ddb34ebe..49a8d61b938027 100644
--- a/libc/config/linux/riscv/entrypoints.txt
+++ b/libc/config/linux/riscv/entrypoints.txt
@@ -631,6 +631,7 @@ if(LIBC_TYPES_HAS_CFLOAT128)
 endif()
 
 if(LIBC_TYPES_HAS_FLOAT128)    
+  list(APPEND TARGET_LIBM_ENTRYPOINTS
     # math.h C23 _Float128 entrypoints
     libc.src.math.canonicalizef128
     libc.src.math.ceilf128

@Sh0g0-1758 Sh0g0-1758 merged commit 9f75b66 into llvm:main Dec 26, 2024
11 of 13 checks passed
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.

2 participants