Skip to content

Reapply "[libc] printf, putchar and vprintf in bareemetal entrypoints… #95619

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
Jun 14, 2024

Conversation

zeroomega
Copy link
Contributor

This reverts commit eca988a. The underlying libc issue was fixed by PR#95576.

The original PR is #95436 , which adds printf, putchar and vprintf in bareemetal entrypoints

…llvm#95436)"

This reverts commit eca988a.
The underlying libc issue was fixed by PR#95576.
@zeroomega zeroomega requested a review from petrhosek June 14, 2024 22:53
@llvmbot llvmbot added the libc label Jun 14, 2024
@llvmbot
Copy link
Member

llvmbot commented Jun 14, 2024

@llvm/pr-subscribers-libc

Author: Haowei (zeroomega)

Changes

This reverts commit eca988a. The underlying libc issue was fixed by PR#95576.

The original PR is #95436 , which adds printf, putchar and vprintf in bareemetal entrypoints


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

2 Files Affected:

  • (modified) libc/config/baremetal/arm/entrypoints.txt (+3)
  • (modified) libc/config/baremetal/riscv/entrypoints.txt (+3)
diff --git a/libc/config/baremetal/arm/entrypoints.txt b/libc/config/baremetal/arm/entrypoints.txt
index 7fb82c60a1bb8..2930d718fdb21 100644
--- a/libc/config/baremetal/arm/entrypoints.txt
+++ b/libc/config/baremetal/arm/entrypoints.txt
@@ -80,8 +80,11 @@ set(TARGET_LIBC_ENTRYPOINTS
 
     # stdio.h entrypoints
     libc.src.stdio.remove
+    libc.src.stdio.printf
+    libc.src.stdio.putchar
     libc.src.stdio.sprintf
     libc.src.stdio.snprintf
+    libc.src.stdio.vprintf
     libc.src.stdio.vsprintf
     libc.src.stdio.vsnprintf
 
diff --git a/libc/config/baremetal/riscv/entrypoints.txt b/libc/config/baremetal/riscv/entrypoints.txt
index b769b43f03a2c..6d38676b1e8a8 100644
--- a/libc/config/baremetal/riscv/entrypoints.txt
+++ b/libc/config/baremetal/riscv/entrypoints.txt
@@ -80,8 +80,11 @@ set(TARGET_LIBC_ENTRYPOINTS
 
     # stdio.h entrypoints
     libc.src.stdio.remove
+    libc.src.stdio.printf
+    libc.src.stdio.putchar
     libc.src.stdio.sprintf
     libc.src.stdio.snprintf
+    libc.src.stdio.vprintf
     libc.src.stdio.vsprintf
     libc.src.stdio.vsnprintf
 

@zeroomega zeroomega merged commit 98b117e into llvm:main Jun 14, 2024
6 of 7 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.

4 participants