This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Remove std::function in C++03
ClosedPublic

Authored by philnik on Oct 13 2022, 4:56 AM.

Details

Reviewers
ldionne
Mordante
Group Reviewers
Restricted Project
Commits
rGdbe60caa922d: [libc++] Remove std::function in C++03
Summary

We've said that we'll remove std::function from C++03 in LLVM 16, so we might as well do it now before we forget.

Diff Detail

Event Timeline

philnik created this revision.Oct 13 2022, 4:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 13 2022, 4:56 AM
philnik requested review of this revision.Oct 13 2022, 4:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 13 2022, 4:56 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript

Please add an entry to the release notes.
Please add in the commit message why you want to remove it now.

philnik updated this revision to Diff 468371.Oct 17 2022, 4:54 PM
  • Address comments
philnik edited the summary of this revision. (Show Details)Oct 17 2022, 4:56 PM
jloser added a subscriber: jloser.Oct 17 2022, 5:52 PM

LGTM for what it's worth. Thanks for the cleanup.

ldionne accepted this revision.Oct 18 2022, 7:52 AM
ldionne added inline comments.
libcxx/docs/ReleaseNotes.rst
90
This revision is now accepted and ready to land.Oct 18 2022, 7:52 AM
Mordante accepted this revision.Oct 18 2022, 10:17 AM

Thanks for the rationale! LGTM!

This revision was automatically updated to reflect the committed changes.
philnik marked an inline comment as done.
libcxx/include/__functional/function.h