This is an archive of the discontinued LLVM Phabricator instance.

[ADT] Deprecate Optional::getPointer
ClosedPublic

Authored by kazu on Nov 23 2022, 3:24 PM.

Details

Summary

This patch deprecates Optional::getPointer. Note that I've converted
all known occurrences of X.getPointer() to &*X on Nov 21, 2022 in
commit 1f914944b6c9a5e4229ceb9f06140fdf178c5ea0.

This is part of an effort to migrate from llvm::Optional to
std::optional:

https://discourse.llvm.org/t/deprecating-llvm-optional-x-hasvalue-getvalue-getvalueor/63716

Diff Detail

Event Timeline

kazu created this revision.Nov 23 2022, 3:24 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 23 2022, 3:24 PM
kazu requested review of this revision.Nov 23 2022, 3:24 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 23 2022, 3:24 PM
MaskRay accepted this revision.Nov 23 2022, 4:07 PM
This revision is now accepted and ready to land.Nov 23 2022, 4:07 PM
This revision was landed with ongoing or failed builds.Nov 23 2022, 4:50 PM
This revision was automatically updated to reflect the committed changes.