This is an archive of the discontinued LLVM Phabricator instance.

[ADT] Use std::nullopt instead of llvm::None in OptionalTest.cpp
ClosedPublic

Authored by kazu on Dec 7 2022, 12:04 AM.

Details

Summary

This patch updates the test for the streaming operator for
llvm::Optional to use std::nullopt instead of llvm::None.

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.Dec 7 2022, 12:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 7 2022, 12:04 AM
kazu requested review of this revision.Dec 7 2022, 12:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 7 2022, 12:04 AM
MaskRay accepted this revision.Dec 7 2022, 6:37 AM
This revision is now accepted and ready to land.Dec 7 2022, 6:37 AM
This revision was landed with ongoing or failed builds.Dec 8 2022, 10:07 AM
This revision was automatically updated to reflect the committed changes.