Skip to content

[libc] Add link to stdfix.h on headers page #129138

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

Merged
merged 2 commits into from
Mar 6, 2025
Merged

Conversation

PiJoules
Copy link
Contributor

No description provided.

@PiJoules PiJoules requested review from petrhosek and lntue February 27, 2025 22:42
@llvmbot llvmbot added the libc label Feb 27, 2025
@llvmbot
Copy link
Member

llvmbot commented Feb 27, 2025

@llvm/pr-subscribers-libc

Author: None (PiJoules)

Changes

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

1 Files Affected:

  • (modified) libc/docs/headers/index.rst (+1)
diff --git a/libc/docs/headers/index.rst b/libc/docs/headers/index.rst
index 3dc30ef90a8e4..e569a22d53a93 100644
--- a/libc/docs/headers/index.rst
+++ b/libc/docs/headers/index.rst
@@ -23,6 +23,7 @@ Implementation Status
    setjmp
    signal
    stdbit
+   stdfix <math/stdfix.rst>
    stdio
    stdlib
    string

@@ -23,6 +23,7 @@ Implementation Status
setjmp
signal
stdbit
stdfix <math/stdfix.rst>
Copy link
Member

Choose a reason for hiding this comment

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

I'd move stdfix.rst to top-level to match other headers.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Like this?

@PiJoules PiJoules merged commit 1a09ada into llvm:main Mar 6, 2025
12 of 16 checks passed
@PiJoules PiJoules deleted the stdfix-link branch March 6, 2025 18:31
boomanaiden154 added a commit to boomanaiden154/llvm-project that referenced this pull request Mar 6, 2025
The docs build action was failing with libc due to checks.rst not
existing in the expected path. This patch adjusts the path to the actual
path which seems to make everything happy. It seems like this did not
show up before as stdfix.rst was not included in a place that actually
caused it to get picked up by sphinx.
boomanaiden154 added a commit that referenced this pull request Mar 6, 2025
The docs build action was failing with libc due to checks.rst not
existing in the expected path. This patch adjusts the path to the actual
path which seems to make everything happy. It seems like this did not
show up before as stdfix.rst was not included in a place that actually
caused it to get picked up by sphinx.
jph-13 pushed a commit to jph-13/llvm-project that referenced this pull request Mar 21, 2025
jph-13 pushed a commit to jph-13/llvm-project that referenced this pull request Mar 21, 2025
The docs build action was failing with libc due to checks.rst not
existing in the expected path. This patch adjusts the path to the actual
path which seems to make everything happy. It seems like this did not
show up before as stdfix.rst was not included in a place that actually
caused it to get picked up by sphinx.
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.

4 participants