This is an archive of the discontinued LLVM Phabricator instance.

[libc++] add global variable template std::views::empty
ClosedPublic

Authored by huixie90 on Apr 3 2022, 8:49 AM.

Details

Summary

[libc++] add global variable template std::views::empty
Note it is neither a range adaptor, nor a CPO. It is simplify a global variable template.

Diff Detail

Event Timeline

huixie90 created this revision.Apr 3 2022, 8:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 3 2022, 8:49 AM
huixie90 requested review of this revision.Apr 3 2022, 8:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 3 2022, 8:49 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision.Apr 4 2022, 10:19 AM
Mordante added a subscriber: Mordante.

LGTM!

libcxx/include/ranges
124

Nice catch!

This revision is now accepted and ready to land.Apr 4 2022, 10:19 AM

I do not have access to push the patch to the remote. If this looks good to you, could you help to commit this.
Author: Hui Xie <hui.xie1990@gmail.com>

This revision was landed with ongoing or failed builds.Apr 5 2022, 9:18 AM
This revision was automatically updated to reflect the committed changes.

I've landed the patch for you.