Skip to content

Conversation

aaryanshukla
Copy link
Contributor

No description provided.

@llvmbot llvmbot added the libc label Aug 1, 2024
@llvmbot
Copy link
Member

llvmbot commented Aug 1, 2024

@llvm/pr-subscribers-libc

Author: None (aaryanshukla)

Changes

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

1 Files Affected:

  • (modified) libc/config/linux/arm/entrypoints.txt (-1)
diff --git a/libc/config/linux/arm/entrypoints.txt b/libc/config/linux/arm/entrypoints.txt
index a9f4383542fe7..90aae962080cd 100644
--- a/libc/config/linux/arm/entrypoints.txt
+++ b/libc/config/linux/arm/entrypoints.txt
@@ -228,7 +228,6 @@ set(TARGET_LIBM_ENTRYPOINTS
     libc.src.math.cos
     libc.src.math.cosf
     libc.src.math.coshf
-    libc.src.math.daddl
     libc.src.math.dfmal
     libc.src.math.dsqrtl
     libc.src.math.erff

Copy link
Contributor

@RoseZhang03 RoseZhang03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aaryanshukla aaryanshukla changed the title removing daddl from arm32 [libc][math][C23] removing daddl from arm32 Aug 1, 2024
Copy link
Contributor

@michaelrj-google michaelrj-google left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might be better to just comment it out instead of fully removing it from the list, but LGTM either way

@aaryanshukla aaryanshukla merged commit 7471387 into llvm:main Aug 1, 2024
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.

4 participants