[libcxx-commits] [PATCH] D152297: [libc++][spaceship][NFC] Updates `SpaceshipProjects.csv` with full details from `P1614R2`

Hristo Hristov via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Jun 8 00:09:20 PDT 2023


H-G-Hristov added a comment.

In D152297#4404128 <https://reviews.llvm.org/D152297#4404128>, @Mordante wrote:

> Thanks for working on this! I want to look at it in more detail later this week. I want to make sure we have all missing parts of the paper listed.

To clarify my motivation for the current format: As I am not that familiar with the structure of the format to make the section of the paper easier to track I decided to add all sections of the standard, which were changed in the paper to the paper status list. Sometimes some items (e.g. ".syn") contain all requirements and sometimes the implementation details are given in a separate section. I added all of them. I have not updated these extra sections yet.
My plan is for me (or if somebody else chips in) is to start from the top of the list check the implementation mark everything which is done and implement the missing parts in subsequent series of patches.

Please note some sections were removed or renamed by subsequent papers. Some sections are probably just an improvement to the wording in standard.



================
Comment at: libcxx/docs/Status/SpaceshipPapers.csv:3
 `P1614R2 <https://wg21.link/P1614>`_,The Mothership has Landed,|In Progress|,
-`P2404R0 <https://wg21.link/P2404>`_,"Relaxing equality_comparable_with's, totally_ordered_with's, and three_way_comparable_with's common reference requirements to support move-only types",,
+`P2404R3 <https://wg21.link/P2404>`_,"Relaxing equality_comparable_with's, totally_ordered_with's, and three_way_comparable_with's common reference requirements to support move-only types",,
 `LWG3330 <https://wg21.link/LWG3330>`_,Include <compare> from most library headers,"|Complete|","13.0"
----------------
Mordante wrote:
> 
I noticed this but it seems to work as expected and redirect to the most current version so I left it unchanged. Do you still want me to update it?

PS. DONE


================
Comment at: libcxx/docs/Status/SpaceshipProjects.csv:189
+| `[stacktrace.basic.cmp] <https://wg21.link/stacktrace.basic.cmp>`_,| `basic_stacktrace <https://reviews.llvm.org/D123228>`_,[alg.three.way],Nikolas Klauser,|In Progress|
+`wg21.link - WG21 redirect service <https://wg21.link>`_,-,-,-,-
----------------
Mordante wrote:
> This seems like an unwanted left-over.
Ah, sorry! I meant to remove it! Thanks for noticing it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D152297/new/

https://reviews.llvm.org/D152297



More information about the libcxx-commits mailing list