[libc-commits] [libc] [libc][docs] Generate docs for signal.h & optimized is_implemented func (PR #88028)

Xu Zhang via libc-commits libc-commits at lists.llvm.org
Mon Apr 8 19:42:51 PDT 2024


https://github.com/simonzgx updated https://github.com/llvm/llvm-project/pull/88028

>From 4a1d17a519d5177c06e5a619a338829d5efd9c91 Mon Sep 17 00:00:00 2001
From: Xu Zhang <simonzgx at gmail.com>
Date: Tue, 9 Apr 2024 03:25:38 +0800
Subject: [PATCH 1/4] [libc][docs] Generate docs for signal.h & optimized
 docgen.py's is_implemented func. (#87835)

---
 libc/docs/signal.rst          | 41 +++++++++++++++++++++++++++++++++++
 libc/utils/docgen/docgen.py   | 17 +++++++++++----
 libc/utils/docgen/signal.json | 29 +++++++++++++++++++++++++
 3 files changed, 83 insertions(+), 4 deletions(-)
 create mode 100644 libc/docs/signal.rst
 create mode 100644 libc/utils/docgen/signal.json

diff --git a/libc/docs/signal.rst b/libc/docs/signal.rst
new file mode 100644
index 00000000000000..a994645ce3c260
--- /dev/null
+++ b/libc/docs/signal.rst
@@ -0,0 +1,41 @@
+signal.h Functions
+==================
+
+.. list-table::
+  :widths: auto
+  :align: center
+  :header-rows: 1
+
+  * - Function
+    - Implemented
+    - Standard
+  * - kill
+    - |check|
+    -
+  * - raise
+    - |check|
+    - 7.14.2.1
+  * - sigaction
+    - |check|
+    -
+  * - sigaddset
+    - |check|
+    -
+  * - sigaltstack
+    - |check|
+    -
+  * - sigdelset
+    - |check|
+    -
+  * - sigemptyset
+    - |check|
+    -
+  * - sigfillset
+    - |check|
+    -
+  * - signal
+    - |check|
+    - 7.14.1.1
+  * - sigprocmask
+    - |check|
+    -
diff --git a/libc/utils/docgen/docgen.py b/libc/utils/docgen/docgen.py
index 7411b4506f082f..213ed7e6cc5ec8 100755
--- a/libc/utils/docgen/docgen.py
+++ b/libc/utils/docgen/docgen.py
@@ -23,12 +23,21 @@ def load_api(hname: str) -> Dict:
 # TODO: we may need to get more sophisticated for less generic implementations.
 # Does libc/src/{hname minus .h suffix}/{fname}.cpp exist?
 def is_implemented(hname: str, fname: str) -> bool:
-    return Path(
+    path = Path(
         Path(__file__).parent.parent.parent,
         "src",
-        hname.rstrip(".h"),
-        fname + ".cpp",
-    ).exists()
+        hname.rstrip(".h")
+    )
+    source_file_name = fname + ".cpp"
+    if path.joinpath(source_file_name).exists():
+        return True
+    
+    # Attempt to search in the subfolders with 1-depth.
+    for sub_dir in path.iterdir():
+        if (sub_dir.joinpath(source_file_name)).exists():
+            return True
+
+    return False
 
 
 def print_functions(header: str, functions: Dict):
diff --git a/libc/utils/docgen/signal.json b/libc/utils/docgen/signal.json
new file mode 100644
index 00000000000000..d7e73fc29fcd56
--- /dev/null
+++ b/libc/utils/docgen/signal.json
@@ -0,0 +1,29 @@
+{
+  "macros": [
+    "SIG_DFL",
+    "SIG_ERR",
+    "SIG_IGN",
+    "SIGABRT",
+    "SIGFPE",
+    "SIGILL",
+    "SIGINT",
+    "SIGSEGV",
+    "SIGTERM"
+  ],
+  "functions": {
+    "kill": null,
+    "sigaction": null,
+    "sigaddset": null,
+    "sigaltstack": null,
+    "sigdelset": null,
+    "sigemptyset": null,
+    "sigfillset": null,
+    "sigprocmask": null,
+    "signal": {
+      "defined": "7.14.1.1"
+    },
+    "raise": {
+      "defined": "7.14.2.1"
+    }
+  }
+}
\ No newline at end of file

>From ed2f221d278ce46283d1f011e3ea71e23dec134a Mon Sep 17 00:00:00 2001
From: Xu Zhang <simonzgx at gmail.com>
Date: Tue, 9 Apr 2024 03:46:40 +0800
Subject: [PATCH 2/4] [libc][docs] Generate docs for signal.h & optimized
 docgen.py's is_implemented func. (#87835)

---
 libc/utils/docgen/docgen.py | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/libc/utils/docgen/docgen.py b/libc/utils/docgen/docgen.py
index 213ed7e6cc5ec8..20edadf7621ca5 100755
--- a/libc/utils/docgen/docgen.py
+++ b/libc/utils/docgen/docgen.py
@@ -29,12 +29,10 @@ def is_implemented(hname: str, fname: str) -> bool:
         hname.rstrip(".h")
     )
     source_file_name = fname + ".cpp"
-    if path.joinpath(source_file_name).exists():
-        return True
-    
-    # Attempt to search in the subfolders with 1-depth.
-    for sub_dir in path.iterdir():
-        if (sub_dir.joinpath(source_file_name)).exists():
+    # Recursively search for the target source file in the subdirectories under
+    # libc/src/{hname}.
+    for sub_dir in path.glob("**"):
+        if sub_dir.joinpath(source_file_name).exists():
             return True
 
     return False

>From 29f6b684f45fb4757cd07eb1a446e9a1e8d318ad Mon Sep 17 00:00:00 2001
From: Xu Zhang <simonzgx at gmail.com>
Date: Tue, 9 Apr 2024 09:52:54 +0800
Subject: [PATCH 3/4] fix missing include in signal.rst and amend
 is_implemented func

---
 libc/docs/signal.rst          | 2 ++
 libc/utils/docgen/docgen.py   | 6 ++----
 libc/utils/docgen/signal.json | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/libc/docs/signal.rst b/libc/docs/signal.rst
index a994645ce3c260..7903bb439cb337 100644
--- a/libc/docs/signal.rst
+++ b/libc/docs/signal.rst
@@ -1,3 +1,5 @@
+.. include:: check.rst
+
 signal.h Functions
 ==================
 
diff --git a/libc/utils/docgen/docgen.py b/libc/utils/docgen/docgen.py
index 20edadf7621ca5..36eb409421b4f5 100755
--- a/libc/utils/docgen/docgen.py
+++ b/libc/utils/docgen/docgen.py
@@ -28,12 +28,10 @@ def is_implemented(hname: str, fname: str) -> bool:
         "src",
         hname.rstrip(".h")
     )
-    source_file_name = fname + ".cpp"
     # Recursively search for the target source file in the subdirectories under
     # libc/src/{hname}.
-    for sub_dir in path.glob("**"):
-        if sub_dir.joinpath(source_file_name).exists():
-            return True
+    for _ in path.glob("**/" + fname + ".cpp"):
+        return True
 
     return False
 
diff --git a/libc/utils/docgen/signal.json b/libc/utils/docgen/signal.json
index d7e73fc29fcd56..976021a803a672 100644
--- a/libc/utils/docgen/signal.json
+++ b/libc/utils/docgen/signal.json
@@ -26,4 +26,4 @@
       "defined": "7.14.2.1"
     }
   }
-}
\ No newline at end of file
+}

>From 9a82d2b4c44d4af1c03fb9782e3cb112b0887585 Mon Sep 17 00:00:00 2001
From: Xu Zhang <simonzgx at gmail.com>
Date: Tue, 9 Apr 2024 10:41:13 +0800
Subject: [PATCH 4/4] update sphinx index file

---
 libc/docs/index.rst | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libc/docs/index.rst b/libc/docs/index.rst
index 8470c8d9287c2f..11d5ae197d7189 100644
--- a/libc/docs/index.rst
+++ b/libc/docs/index.rst
@@ -70,6 +70,7 @@ stages there is no ABI stability in any form.
    libc_search
    c23
    ctype
+   signal
 
 .. toctree::
    :hidden:



More information about the libc-commits mailing list