-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[libc][docs] Add sys/statvfs to documentation and YAML definitions #126413
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
Conversation
@llvm/pr-subscribers-libc Author: Prashanth (StarOne01) ChangesThese changes ensure that the sys/statvfs header is documented properly with respect to the issue ( #122006 ) . cc @nickdesaulniers Full diff: https://github.com/llvm/llvm-project/pull/126413.diff 3 Files Affected:
diff --git a/libc/docs/CMakeLists.txt b/libc/docs/CMakeLists.txt
index 97f27fe6a6e0c6b..4ef0e920de683d3 100644
--- a/libc/docs/CMakeLists.txt
+++ b/libc/docs/CMakeLists.txt
@@ -59,6 +59,7 @@ if (SPHINX_FOUND)
sys/mman
sys/resource
sys/stat
+ sys/statvfs
sys/time
sys/wait
termios
diff --git a/libc/docs/headers/index.rst b/libc/docs/headers/index.rst
index 745b6f44750dbcf..3dc30ef90a8e4cb 100644
--- a/libc/docs/headers/index.rst
+++ b/libc/docs/headers/index.rst
@@ -30,6 +30,7 @@ Implementation Status
sys/mman
sys/resource
sys/stat
+ sys/statvfs
sys/time
sys/wait
termios
diff --git a/libc/utils/docgen/sys/statvfs.yaml b/libc/utils/docgen/sys/statvfs.yaml
new file mode 100644
index 000000000000000..43fb92166f8a02c
--- /dev/null
+++ b/libc/utils/docgen/sys/statvfs.yaml
@@ -0,0 +1,11 @@
+functions:
+ fstatvfs:
+ in-latest-posix: ''
+ statvfs:
+ in-latest-posix: ''
+
+macros:
+ ST_RDONLY:
+ in-latest-posix: ''
+ ST_NOSUID:
+ in-latest-posix: ''
\ No newline at end of file
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Consider adding a newline to the yaml file at the end. Need us to merge this for you? Thanks for the patch.
Done. Yesss pleasee! |
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/73/builds/12983 Here is the relevant piece of the build log for the reference
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/153/builds/22451 Here is the relevant piece of the build log for the reference
|
Umm... Are this related to this commit @nickdesaulniers ? |
No |
…lvm#126413) These changes ensure that the sys/statvfs header is documented properly with respect to the issue ( llvm#122006 ) .
…lvm#126413) These changes ensure that the sys/statvfs header is documented properly with respect to the issue ( llvm#122006 ) .
These changes ensure that the sys/statvfs header is documented properly with respect to the issue ( #122006 ) .
cc @nickdesaulniers