This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Add FTM for views::as_rvalue
ClosedPublic

Authored by philnik on Feb 25 2023, 1:28 AM.

Details

Reviewers
ldionne
Mordante
Group Reviewers
Restricted Project
Commits
rG3d4b7a6fa8a4: [libc++] Add FTM for views::as_rvalue
Summary

views::as_rvalue was implemented in D137637, but we forgot to set the feature test macro.
Fixes #60986.

Diff Detail

Event Timeline

philnik created this revision.Feb 25 2023, 1:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 25 2023, 1:28 AM
Herald added a subscriber: arichardson. · View Herald Transcript
philnik requested review of this revision.Feb 25 2023, 1:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 25 2023, 1:28 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik edited the summary of this revision. (Show Details)Feb 25 2023, 1:28 AM

LGTM!

Since this paper landed in libc++ 16 I would like to backport this patch to the LLVM 16 branch.

Mordante accepted this revision.Feb 25 2023, 5:46 AM
This revision is now accepted and ready to land.Feb 25 2023, 5:46 AM
This revision was landed with ongoing or failed builds.Feb 25 2023, 9:27 AM
This revision was automatically updated to reflect the committed changes.