This is an archive of the discontinued LLVM Phabricator instance.

[ADT] Deprecate Optional::map
ClosedPublic

Authored by kazu on Aug 13 2022, 12:07 PM.

Details

Summary

This patch deprecates Optional::map in favor of Optional::transform
for consistency with std::optional::transform in C++23.

Note that I've migrated all known users of Optional::map.

Diff Detail

Event Timeline

kazu created this revision.Aug 13 2022, 12:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 13 2022, 12:07 PM
kazu requested review of this revision.Aug 13 2022, 12:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 13 2022, 12:07 PM
dblaikie accepted this revision.Aug 14 2022, 1:39 PM

Sounds good!

This revision is now accepted and ready to land.Aug 14 2022, 1:39 PM
This revision was landed with ongoing or failed builds.Aug 14 2022, 5:52 PM
This revision was automatically updated to reflect the committed changes.