Skip to content

Conversation

zimirza
Copy link
Contributor

@zimirza zimirza commented Aug 31, 2025

I noticed that I forgot to add ctime and ctime_r to docs in #107285. This is a simple patch to add it.

@llvmbot llvmbot added the libc label Aug 31, 2025
@llvmbot
Copy link
Member

llvmbot commented Aug 31, 2025

@llvm/pr-subscribers-libc

Author: Zishan Mirza (zimirza)

Changes

I noticed that I forgot to add ctime and ctime_r to docs in #107285. This is a simple patch to add it.


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

1 Files Affected:

  • (modified) libc/docs/headers/time.rst (+2-2)
diff --git a/libc/docs/headers/time.rst b/libc/docs/headers/time.rst
index 9733a176fbb25..934a8a7cf22cc 100644
--- a/libc/docs/headers/time.rst
+++ b/libc/docs/headers/time.rst
@@ -73,9 +73,9 @@ Implementation Status
 +---------------------+---------+---------+---------+-----------------+---------+---------+---------+---------+---------+---------+---------+---------+---------+
 | clock_settime       |         |         |         |                 |         |         |         |         |         |         |         |         |         |
 +---------------------+---------+---------+---------+-----------------+---------+---------+---------+---------+---------+---------+---------+---------+---------+
-| ctime               |         |         |         |                 |         |         |         |         |         |         |         |         |         |
+| ctime               | |check| | |check| |         |     |check|     |         |         |         |         |         |         |         |         |         |
 +---------------------+---------+---------+---------+-----------------+---------+---------+---------+---------+---------+---------+---------+---------+---------+
-| ctime_r             |         |         |         |                 |         |         |         |         |         |         |         |         |         |
+| ctime_r             | |check| | |check| |         |     |check|     |         |         |         |         |         |         |         |         |         |
 +---------------------+---------+---------+---------+-----------------+---------+---------+---------+---------+---------+---------+---------+---------+---------+
 | difftime            | |check| | |check| |         |     |check|     |         |         |         |         |         |         |         |         |         |
 +---------------------+---------+---------+---------+-----------------+---------+---------+---------+---------+---------+---------+---------+---------+---------+

Copy link
Contributor

@michaelrj-google michaelrj-google left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, do you need me to merge this?

@zimirza
Copy link
Contributor Author

zimirza commented Sep 5, 2025

That would be great. Thank you.

@michaelrj-google michaelrj-google merged commit b946b2b into llvm:main Sep 5, 2025
22 checks passed
@zimirza zimirza deleted the ctime_update_docs branch September 5, 2025 18:01
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