This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Add Implemented Papers section
ClosedPublic

Authored by philnik on Jun 13 2022, 10:58 AM.

Details

Reviewers
ldionne
Mordante
var-const
Group Reviewers
Restricted Project
Commits
rG3766992291fd: [libc++] Add Implemented Papers section

Diff Detail

Event Timeline

philnik created this revision.Jun 13 2022, 10:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 13 2022, 10:58 AM
philnik requested review of this revision.Jun 13 2022, 10:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 13 2022, 10:58 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript

I'm not convinced this change is really needed, but I'm not objecting either.
However when we move in this direction, lets move all paper related changes to this section.
(I expect some thing migh be ambiguous, but I marked the clear ones.)

libcxx/docs/ReleaseNotes.rst
47

Please move these to the new section too. Just move them as-is.

76–77

Please move these to the new section too. Please leave the additional information only strip the Implemented.

philnik updated this revision to Diff 437979.Jun 17 2022, 11:21 AM
philnik marked 2 inline comments as done.
  • Address comments
Mordante accepted this revision.Jun 17 2022, 12:55 PM

LGTM, but please give other reviewers a bit more time to have a look.

libcxx/docs/ReleaseNotes.rst
78

Actually I meant to move this entire paragraph to Implemented Papers, but this is fine by me too.

This revision is now accepted and ready to land.Jun 17 2022, 12:55 PM
ldionne accepted this revision.Jun 17 2022, 1:42 PM
This revision was automatically updated to reflect the committed changes.