Skip to content

[libc] Remove unnecessary stdio.h includes #98132

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 3 commits into from
Jul 9, 2024

Conversation

petrhosek
Copy link
Member

@petrhosek petrhosek commented Jul 9, 2024

This are unused.

@llvmbot llvmbot added the libc label Jul 9, 2024
@llvmbot
Copy link
Member

llvmbot commented Jul 9, 2024

@llvm/pr-subscribers-libc

Author: Petr Hosek (petrhosek)

Changes

This is unused.


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

1 Files Affected:

  • (modified) libc/src/stdio/puts.h (-2)
diff --git a/libc/src/stdio/puts.h b/libc/src/stdio/puts.h
index 11fbd0be897cf..68d5225b00a17 100644
--- a/libc/src/stdio/puts.h
+++ b/libc/src/stdio/puts.h
@@ -9,8 +9,6 @@
 #ifndef LLVM_LIBC_SRC_STDIO_PUTS_H
 #define LLVM_LIBC_SRC_STDIO_PUTS_H
 
-#include <stdio.h>
-
 namespace LIBC_NAMESPACE {
 
 int puts(const char *__restrict str);

@petrhosek petrhosek changed the title [libc] Remove the stdio.h include from puts.h [libc] Remove unnecessary stdio.h includes Jul 9, 2024
@SchrodingerZhu SchrodingerZhu merged commit c530dad into llvm:main Jul 9, 2024
6 checks passed
aaryanshukla pushed a commit to aaryanshukla/llvm-project that referenced this pull request Jul 14, 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