This is an archive of the discontinued LLVM Phabricator instance.

[libc++][doc] Update paper status.
ClosedPublic

Authored by Mordante on Oct 5 2021, 9:19 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rG352c3af3ab19: [libc++][doc] Update paper status.
Summary

Update the status with the approved papers and LWG-issues in the October 2021 plenary.

Diff Detail

Event Timeline

Mordante requested review of this revision.Oct 5 2021, 9:19 AM
Mordante created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptOct 5 2021, 9:19 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante added inline comments.Oct 5 2021, 9:21 AM
libcxx/docs/Status/FormatIssues.csv
7

The status of in progress is correct, I already looked at it while working on P2216.

jloser added a subscriber: jloser.Oct 6 2021, 7:21 AM
jloser added inline comments.
libcxx/docs/Status/Cxx2bPapers.csv
34

Feel free to put this as in progress if you'd like. I have a PR for it at https://reviews.llvm.org/D111197

Mordante added inline comments.Oct 6 2021, 9:34 AM
libcxx/docs/Status/Cxx2bPapers.csv
34

In general we don't use In Progress for the official paper status only for subprojects for aligning the work.
But lets see which patch lands first and then make sure the second one sets the proper state for P2251.

ldionne requested changes to this revision.Oct 6 2021, 10:58 AM
ldionne added a subscriber: ldionne.

We seem to be missing the LWG issues that were moved in the meeting. They are in http://wg21.link/p2450r0.

libcxx/docs/Status/Cxx20Papers.csv
203

This is a dupe with the previous line.

libcxx/docs/Status/Cxx2bPapers.csv
34

Yeah let's just rebase @jloser 's patch on top of this one and mark the paper as complete.

This revision now requires changes to proceed.Oct 6 2021, 10:58 AM
Mordante marked 3 inline comments as done.Oct 6 2021, 12:18 PM
Mordante updated this revision to Diff 377638.Oct 6 2021, 12:20 PM

Addresses review comments.
Update the LWG issues.

Mordante edited the summary of this revision. (Show Details)Oct 6 2021, 12:22 PM
ldionne accepted this revision.Oct 8 2021, 10:23 AM

Thanks!

Ideally, we'd have a way to avoid duplicating issues in e.g. Cxx20Issues.csv and RangesIssues.csv by having some sort of tagging system, but csv isn't really the best tool for that.

This revision is now accepted and ready to land.Oct 8 2021, 10:23 AM

As discussed on Discord I want to look at removing the duplication, but in a separate patch.

This revision was automatically updated to reflect the committed changes.