Skip to content

Conversation

frederick-vs-ja
Copy link
Contributor

P3379R0 was implemented in LLVM 21 (13c464b) but the entry for release note is still missing. A following-up PR (4a509f8) fixed this as drive-by but backport was not accepted.

This patch only adds the missing entry.

@frederick-vs-ja frederick-vs-ja added this to the LLVM 21.x Release milestone Oct 16, 2025
@frederick-vs-ja frederick-vs-ja requested a review from a team as a code owner October 16, 2025 11:51
@frederick-vs-ja frederick-vs-ja added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Oct 16, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in LLVM Release Status Oct 16, 2025
@llvmbot
Copy link
Member

llvmbot commented Oct 16, 2025

@llvm/pr-subscribers-libcxx

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

Changes

P3379R0 was implemented in LLVM 21 (13c464b) but the entry for release note is still missing. A following-up PR (4a509f8) fixed this as drive-by but backport was not accepted.

This patch only adds the missing entry.


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

1 Files Affected:

  • (modified) libcxx/docs/ReleaseNotes/21.rst (+1)
diff --git a/libcxx/docs/ReleaseNotes/21.rst b/libcxx/docs/ReleaseNotes/21.rst
index 1410223d56a6f..d6765eb62e4b0 100644
--- a/libcxx/docs/ReleaseNotes/21.rst
+++ b/libcxx/docs/ReleaseNotes/21.rst
@@ -53,6 +53,7 @@ Implemented Papers
 - P2711R1: Making multi-param constructors of ``views`` ``explicit`` (`Github <https://github.com/llvm/llvm-project/issues/105252>`__)
 - P2770R0: Stashing stashing ``iterators`` for proper flattening (`Github <https://github.com/llvm/llvm-project/issues/105250>`__)
 - P2655R3: ``common_reference_t`` of ``reference_wrapper`` Should Be a Reference Type (`Github <https://github.com/llvm/llvm-project/issues/105260>`__)
+- P3379R0: Constrain ``std::expected equality`` operators (`Github <https://github.com/llvm/llvm-project/issues/118135>`__)
 
 Improvements and New Features
 -----------------------------

@github-project-automation github-project-automation bot moved this from Needs Triage to Needs Merge in LLVM Release Status Oct 16, 2025
@Zingam
Copy link
Contributor

Zingam commented Oct 16, 2025

Does this need to be 'backported'?

@frederick-vs-ja frederick-vs-ja force-pushed the llvm-21-release-notes-p3379r0 branch from c2bf947 to d3da6a9 Compare October 17, 2025 01:57
P3379R0 was implemented in LLVM 21 (13c464b) but the entry for release note is still missing. A following-up PR (4a509f8) fixed this as drive-by but backport was not accepted.

This patch only adds the missing entry.
@c-rhodes c-rhodes force-pushed the llvm-21-release-notes-p3379r0 branch from d3da6a9 to c6af6be Compare October 17, 2025 07:44
@c-rhodes c-rhodes merged commit c6af6be into llvm:release/21.x Oct 17, 2025
1 check was pending
@github-project-automation github-project-automation bot moved this from Needs Merge to Done in LLVM Release Status Oct 17, 2025
Copy link

@frederick-vs-ja (or anyone else). If you would like to add a note about this fix in the release notes (completely optional). Please reply to this comment with a one or two sentence description of the fix. When you are done, please add the release:note label to this PR.

@frederick-vs-ja frederick-vs-ja deleted the llvm-21-release-notes-p3379r0 branch October 17, 2025 07:56
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

Development

Successfully merging this pull request may close these issues.

4 participants