This is an archive of the discontinued LLVM Phabricator instance.

[libc++][doc] Adds Issaquah papers and issues.
ClosedPublic

Authored by Mordante on Feb 12 2023, 4:08 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rGf8877d86a867: [libc++][doc] Adds Issaquah papers and issues.

Diff Detail

Event Timeline

Mordante created this revision.Feb 12 2023, 4:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 12 2023, 4:08 AM
Mordante requested review of this revision.Feb 12 2023, 4:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 12 2023, 4:08 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
var-const added inline comments.
libcxx/docs/Status/Cxx2bIssues.csv
279

I think we can mark this one as "Nothing to do" (in this patch or a follow-up), AFAICS it doesn't affect the implementation.

ldionne accepted this revision.Feb 14 2023, 8:45 AM

Let's mark any NFCs as separate patches.

This revision is now accepted and ready to land.Feb 14 2023, 8:45 AM

I pinged some folks on some LWG issues, it would be nice if you could have a look when you have some time.

libcxx/docs/Status/Cxx2bIssues.csv
252
261
292
293
294
295
308
libcxx/docs/Status/Cxx2bPapers.csv
119

@huixie90 This is your paper I think

Mordante marked an inline comment as done.Feb 14 2023, 10:04 AM

Thanks for the reviews!

libcxx/docs/Status/Cxx2bIssues.csv
279

I had a look at the issue and see only removals. I still think it's good to validate whether these lessened restrictions are something we can/want to take advantage of in libc++. Since this is not my area of expertise I prefer to leave it open so a you or another ranges expert can decide whether or not to close it with nothing to do.

This revision was automatically updated to reflect the committed changes.
Mordante marked an inline comment as done.