[libc] corrected yaml files in newhdrgen (llvm#98069)

- fixed little issues in yaml issues through testing - corrected function class in the case there is no attribute
aaryanshukla · Jul 14, 2024 · 80455ad · 80455ad
1 parent 452fdb2
commit 80455ad
Show file tree

Hide file tree

Showing 16 changed files with 149 additions and 159 deletions.
diff --git a/libc/newhdrgen/class_implementation/classes/function.py b/libc/newhdrgen/class_implementation/classes/function.py
@@ -20,7 +20,7 @@ def __init__(
         ]
         self.standards = standards
         self.guard = guard
-        self.attributes = attributes or []
+        self.attributes = attributes or ""
 
     def __str__(self):
         attributes_str = self.attributes

diff --git a/libc/newhdrgen/tests/test_integration.py b/libc/newhdrgen/tests/test_integration.py
@@ -23,7 +23,7 @@ def setUp(self):
         )
 
         self.maxDiff = None
-        # Adjust based on your directory structure such as being in build etc.
+
         self.source_dir = Path(__file__).resolve().parent.parent.parent.parent
 
     def run_script(self, yaml_file, h_def_file, output_dir):

diff --git a/libc/newhdrgen/yaml/inttypes.yaml b/libc/newhdrgen/yaml/inttypes.yaml
@@ -1,5 +1,5 @@
 header: inttypes.h
-macros: 
+macros: []
 types:
   - type_name: imaxdiv_t
 enums: []

diff --git a/libc/newhdrgen/yaml/pthread.yaml b/libc/newhdrgen/yaml/pthread.yaml
@@ -73,23 +73,23 @@ functions:
       - POSIX
     return_type: int
     arguments:
-      - type: const pthread_attr_t * __restrict
-      - type: size_t * __restrict
+      - type: const pthread_attr_t *__restrict
+      - type: size_t *__restrict
   - name: pthread_attr_getstack
     standards: 
       - POSIX
     return_type: int
     arguments:
-      - type: const pthread_attr_t * __restrict
-      - type: void * * __restrict
-      - type: size_t * __restrict
+      - type: const pthread_attr_t *__restrict
+      - type: void **__restrict
+      - type: size_t *__restrict
   - name: pthread_attr_getstacksize
     standards: 
       - POSIX
     return_type: int
     arguments:
-      - type: const pthread_attr_t * __restrict
-      - type: size_t * __restrict
+      - type: const pthread_attr_t *__restrict
+      - type: size_t *__restrict
   - name: pthread_attr_setdetachstate
     standards: 
       - POSIX
@@ -130,15 +130,15 @@ functions:
       - POSIX
     return_type: int
     arguments:
-      - type: const pthread_condattr_t * __restrict
+      - type: const pthread_condattr_t *__restrict
       - type: clockid_t * __restrict
   - name: pthread_condattr_getpshared
     standards: 
       - POSIX
     return_type: int
     arguments:
-      - type: const pthread_condattr_t * __restrict
-      - type: int * __restrict
+      - type: const pthread_condattr_t *__restrict
+      - type: int *__restrict
   - name: pthread_condattr_init
     standards: 
       - POSIX
@@ -164,8 +164,8 @@ functions:
       - POSIX
     return_type: int
     arguments:
-      - type: pthread_t * __restrict
-      - type: const pthread_attr_t * __restrict
+      - type: pthread_t *__restrict
+      - type: const pthread_attr_t *__restrict
       - type: __pthread_start_t
       - type: void *
   - name: pthread_detach
@@ -207,7 +207,7 @@ functions:
     return_type: int
     arguments:
       - type: pthread_t
-      - type: void * *
+      - type: void **
   - name: pthread_key_create
     standards: 
       - POSIX
@@ -244,8 +244,8 @@ functions:
       - POSIX
     return_type: int
     arguments:
-      - type: pthread_mutex_t * __restrict
-      - type: const pthread_mutexattr_t * __restrict
+      - type: pthread_mutex_t *__restrict
+      - type: const pthread_mutexattr_t *__restrict
   - name: pthread_mutex_lock
     standards: 
       - POSIX
@@ -275,42 +275,42 @@ functions:
       - POSIX
     return_type: int
     arguments:
-      - type: const pthread_mutexattr_t * __restrict
-      - type: int * __restrict
+      - type: const pthread_mutexattr_t *__restrict
+      - type: int *__restrict
   - name: pthread_mutexattr_getrobust
     standards: 
       - POSIX
     return_type: int
     arguments:
-      - type: const pthread_mutexattr_t * __restrict
-      - type: int * __restrict
+      - type: const pthread_mutexattr_t *__restrict
+      - type: int *__restrict
   - name: pthread_mutexattr_gettype
     standards: 
       - POSIX
     return_type: int
     arguments:
-      - type: const pthread_mutexattr_t * __restrict
-      - type: int * __restrict
+      - type: const pthread_mutexattr_t *__restrict
+      - type: int *__restrict
   - name: pthread_mutexattr_setpshared
     standards: 
       - POSIX
     return_type: int
     arguments:
-      - type: pthread_mutexattr_t * __restrict
+      - type: pthread_mutexattr_t *__restrict
       - type: int
   - name: pthread_mutexattr_setrobust
     standards: 
       - POSIX
     return_type: int
     arguments:
-      - type: pthread_mutexattr_t * __restrict
+      - type: pthread_mutexattr_t *__restrict
       - type: int
   - name: pthread_mutexattr_settype
     standards: 
       - POSIX
     return_type: int
     arguments:
-      - type: pthread_mutexattr_t * __restrict
+      - type: pthread_mutexattr_t *__restrict
       - type: int
   - name: pthread_once
     standards: 

diff --git a/libc/newhdrgen/yaml/sched.yaml b/libc/newhdrgen/yaml/sched.yaml
@@ -77,7 +77,8 @@ functions:
     standards: 
       - POSIX 
     return_type: int
-    arguments: []
+    arguments: 
+      - type: void
   - name: __sched_getcpucount
     standards: 
       - llvm_libc_ext

diff --git a/libc/newhdrgen/yaml/signal.yaml b/libc/newhdrgen/yaml/signal.yaml
@@ -1,7 +1,5 @@
 header: signal.h
-macros:
-  - macro_name: __need_size_t
-    macro_value: null
+macros: []
 types:
   - type_name: pid_t
   - type_name: stack_t
@@ -32,15 +30,15 @@ functions:
     return_type: int
     arguments:
       - type: int
-      - type: const struct sigaction * __restrict
-      - type: struct sigaction * __restrict
+      - type: const struct sigaction *__restrict
+      - type: struct sigaction *__restrict
   - name: sigaltstack
     standards: 
       - POSIX 
     return_type: int
     arguments:
-      - type: const stack_t * __restrict
-      - type: stack_t * __restrict
+      - type: const stack_t *__restrict
+      - type: stack_t *__restrict
   - name: sigdelset
     standards: 
       - POSIX 
@@ -67,8 +65,8 @@ functions:
     return_type: int
     arguments:
       - type: int
-      - type: const sigset_t * __restrict
-      - type: sigset_t * __restrict
+      - type: const sigset_t *__restrict
+      - type: sigset_t *__restrict
   - name: sigfillset
     standards: 
       - POSIX