This is an archive of the discontinued LLVM Phabricator instance.

[libc++][docs] Add status docs for ranges.
ClosedPublic

Authored by zoecarver on Apr 13 2021, 9:49 AM.

Details

Reviewers
ldionne
cjdb
EricWF
Group Reviewers
Restricted Project
Commits
rG0f0149b8d3be: [libc++][docs] Add status docs for ranges.
Summary

This patch is not yet proposed, it is only for discussing the best way to track the status of this huge project.

The added documents have two tables: 1 containing links to issues and papers related to ranges. And the other contains most of the sections from the One Ranges Proposal, with their dependencies linked. This will allow us to assign work that can be done in parallel.

Diff Detail

Event Timeline

zoecarver created this revision.Apr 13 2021, 9:49 AM
zoecarver requested review of this revision.Apr 13 2021, 9:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 13 2021, 9:49 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.Apr 13 2021, 10:48 AM

LGTM as discussed just now. You might want to update some of the names as @cjdb explained in the meeting.

This revision is now accepted and ready to land.Apr 13 2021, 10:48 AM
cjdb accepted this revision.Apr 13 2021, 11:13 AM

LGTM. You don't necessarily need to apply my comments; they're for tracking so that I remember to mark as WIP or complete before making a final merge.

libcxx/docs/OneRangesProposalStatus.csv
7
8
9
13
14
16
35
36

ranges::begin, ranges::end, ranges::cbegin, ranges::cend: D100255

41

Nit. Otherwise that's great you have done this.

libcxx/docs/RangesStatus.rst
18

Thanks for doing that @cjdb! I'll apply those and fix the names.

zoecarver updated this revision to Diff 337296.Apr 13 2021, 5:56 PM
zoecarver marked an inline comment as done.
  • Apply feedback.
This revision was landed with ongoing or failed builds.Apr 13 2021, 6:00 PM
This revision was automatically updated to reflect the committed changes.