Skip to content
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

[libc++][docs] Remove mis-added entry for P2513R4 #130581

Merged

Conversation

frederick-vs-ja
Copy link
Contributor

P2513R4 neither touched library wording nor required library implementation to change. So it was probably a mistake to list it in libc++'s implementation status table.

Closes #105228.

P2513R4 neither touched library wording nor required library
implementation to change. So it was probably a mistake to list it in
libc++'s implementation status table.
@frederick-vs-ja frederick-vs-ja added documentation libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. labels Mar 10, 2025
@frederick-vs-ja frederick-vs-ja requested a review from a team as a code owner March 10, 2025 11:05
@llvmbot
Copy link
Member

llvmbot commented Mar 10, 2025

@llvm/pr-subscribers-libcxx

Author: A. Jiang (frederick-vs-ja)

Changes

P2513R4 neither touched library wording nor required library implementation to change. So it was probably a mistake to list it in libc++'s implementation status table.

Closes #105228.


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

1 Files Affected:

  • (modified) libcxx/docs/Status/Cxx23Papers.csv (-1)
diff --git a/libcxx/docs/Status/Cxx23Papers.csv b/libcxx/docs/Status/Cxx23Papers.csv
index 0f3db57788b49..2589954729a30 100644
--- a/libcxx/docs/Status/Cxx23Papers.csv
+++ b/libcxx/docs/Status/Cxx23Papers.csv
@@ -82,7 +82,6 @@
 "`P2499R0 <https://wg21.link/P2499R0>`__","``string_view`` range constructor should be ``explicit``","2022-07 (Virtual)","|Complete|","16",""
 "`P2502R2 <https://wg21.link/P2502R2>`__","``std::generator``: Synchronous Coroutine Generator for Ranges","2022-07 (Virtual)","","",""
 "`P2508R1 <https://wg21.link/P2508R1>`__","Exposing ``std::basic-format-string``","2022-07 (Virtual)","|Complete|","15",""
-"`P2513R4 <https://wg21.link/P2513R4>`__","``char8_t`` Compatibility and Portability Fixes","2022-07 (Virtual)","","",""
 "`P2517R1 <https://wg21.link/P2517R1>`__","Add a conditional ``noexcept`` specification to ``std::apply``","2022-07 (Virtual)","|Complete|","3.9",""
 "`P2520R0 <https://wg21.link/P2520R0>`__","``move_iterator`` should be a random access iterator","2022-07 (Virtual)","|Complete|","17","Implemented as a DR in C++20"
 "`P2540R1 <https://wg21.link/P2540R1>`__","Empty Product for certain Views","2022-07 (Virtual)","","",""

@frederick-vs-ja frederick-vs-ja merged commit 83936f5 into llvm:main Mar 10, 2025
85 checks passed
@frederick-vs-ja frederick-vs-ja deleted the remove-libcxx-doc-entry-p2513r4 branch March 10, 2025 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

P2513R4: char8_t Compatibility and Portability Fixes
3 participants