Skip to content

Conversation

@vonosmas
Copy link
Contributor

  • Add a missing dependency to install struct_itimerval.h
  • Add fseeko/ftello declarations to stdio.h

* Add a missing dependency to install struct_itimerval.h
* Add fseeko/ftello declarations to stdio.h
@llvmbot
Copy link
Member

llvmbot commented Oct 15, 2025

@llvm/pr-subscribers-libc

Author: Alexey Samsonov (vonosmas)

Changes
  • Add a missing dependency to install struct_itimerval.h
  • Add fseeko/ftello declarations to stdio.h

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

2 Files Affected:

  • (modified) libc/include/CMakeLists.txt (+2-1)
  • (modified) libc/include/stdio.yaml (+14)
diff --git a/libc/include/CMakeLists.txt b/libc/include/CMakeLists.txt
index afa90e6c8b655..81360aac984e0 100644
--- a/libc/include/CMakeLists.txt
+++ b/libc/include/CMakeLists.txt
@@ -632,8 +632,9 @@ add_header_macro(
   sys/time.h
   DEPENDS
     .llvm_libc_common_h
-    .llvm-libc-types.struct_timeval
     .llvm-libc-macros.sys_time_macros
+    .llvm-libc-types.struct_itimerval
+    .llvm-libc-types.struct_timeval
 )
 
 add_header_macro(
diff --git a/libc/include/stdio.yaml b/libc/include/stdio.yaml
index 2a0c563709984..394437ba3bbcd 100644
--- a/libc/include/stdio.yaml
+++ b/libc/include/stdio.yaml
@@ -197,12 +197,26 @@ functions:
       - type: FILE *
       - type: long
       - type: int
+  - name: fseeko
+    standards:
+      - POSIX
+    return_type: int
+    arguments:
+      - type: FILE *
+      - type: off_t
+      - type: int
   - name: ftell
     standards:
       - stdc
     return_type: long
     arguments:
       - type: FILE *
+  - name: ftello
+    standards:
+      - POSIX
+    return_type: off_t
+    arguments:
+      - type: FILE *
   - name: funlockfile
     standards:
       - POSIX

@vonosmas vonosmas merged commit 36f7f34 into llvm:main Oct 16, 2025
22 checks passed
@vonosmas vonosmas deleted the libc-header-fix branch October 16, 2025 01:06
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