This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Acquire locks on Ranges work
ClosedPublic

Authored by ldionne on Apr 30 2021, 3:04 PM.

Details

Reviewers
None
Group Reviewers
Restricted Project
Commits
rGdf280d1368e4: [libc++] Acquire locks on Ranges work
Summary

This commit acquires locks on a few elements of Ranges to make sure we don't
duplicate work. @zoecarver @cjdb please comment here if you want to acquire
locks of your own (or if you want dibs on one of the two features I'm locking)
and I'll add them.

Diff Detail

Event Timeline

ldionne requested review of this revision.Apr 30 2021, 3:04 PM
ldionne created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptApr 30 2021, 3:04 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
cjdb added a comment.Apr 30 2021, 3:07 PM

I'll take ref_view, views::all, and join_view for now.

I'll take view.interface and you should also mark range.view as done, I think (unless that's already done and the CI just hasn't re-built the docs).

Thanks!

ldionne updated this revision to Diff 342383.May 3 2021, 7:40 AM

Acquire locks for cjdb and zoecarver

This revision was not accepted when it landed; it landed in state Needs Review.May 3 2021, 7:40 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.