This is an archive of the discontinued LLVM Phabricator instance.

[ADT] Remove Optional<T>::create
ClosedPublic

Authored by kazu on Dec 10 2022, 12:52 PM.

Details

Summary

The last use of Optional<T>::create was removed on December 10, 2022
in commit 1cb7fba3e57d6c31f05c7a2bda1634da49dd1d18.

Diff Detail

Event Timeline

kazu created this revision.Dec 10 2022, 12:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 10 2022, 12:52 PM
kazu requested review of this revision.Dec 10 2022, 12:52 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 10 2022, 12:52 PM

https://sourcegraph.com/search?q=context:global+Optional%3C%5B%5E%3E%5D%2B%3E::create+-f:clang/lib+-f:lib/StaticAnalyzer/&patternType=regexp&sm=0

This API seems unused outside llvm-project, so I think it can just be removed instead of going through a deprecated state.

kazu updated this revision to Diff 482166.Dec 12 2022, 9:23 AM

Remove the function instead.

kazu retitled this revision from [ADT] Deprecate Optional<T>::create (NFC) to [ADT] Remove Optional<T>::create.Dec 12 2022, 9:23 AM
kazu edited the summary of this revision. (Show Details)

https://sourcegraph.com/search?q=context:global+Optional%3C%5B%5E%3E%5D%2B%3E::create+-f:clang/lib+-f:lib/StaticAnalyzer/&patternType=regexp&sm=0

This API seems unused outside llvm-project, so I think it can just be removed instead of going through a deprecated state.

OK. I've updated the patch. PTAL. Thanks!

kazu added a comment.Dec 13 2022, 11:59 PM

Friendly ping. I'm going for simple removal as @MaskRay has suggested.

There were so few users within LLVM. Probably there are very few out there among downstream users, too.

MaskRay accepted this revision.Dec 14 2022, 10:25 AM
This revision is now accepted and ready to land.Dec 14 2022, 10:25 AM
This revision was landed with ongoing or failed builds.Dec 14 2022, 10:34 AM
This revision was automatically updated to reflect the committed changes.