caio/clang-p2996
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[lldb] Use std::optional instead of llvm::Optional (NFC)

Browse Source 
This patch replaces (llvm::|)Optional< with std::optional<.  I'll post
a separate patch to clean up the "using" declarations, #include
"llvm/ADT/Optional.h", etc.

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
This commit is contained in:
Kazu Hirata
2023-01-07 14:18:35 -08:00
parent f190ce625a
commit 2fe8327406
387 changed files with 1323 additions and 1324 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
lldb/source/Core/Module.cpp
View File

@@ -1126,7 +1126,7 @@ bool Module::FileHasChanged() const {
}
void Module::ReportWarningOptimization(
llvm::Optional<lldb::user_id_t> debugger_id) {
std::optional<lldb::user_id_t> debugger_id) {
ConstString file_name = GetFileSpec().GetFilename();
if (file_name.IsEmpty())
return;
@@ -1140,7 +1140,7 @@ void Module::ReportWarningOptimization(
}
void Module::ReportWarningUnsupportedLanguage(
LanguageType language, llvm::Optional<lldb::user_id_t> debugger_id) {
LanguageType language, std::optional<lldb::user_id_t> debugger_id) {
StreamString ss;
ss << "This version of LLDB has no plugin for the language \""
<< Language::GetNameForLanguageType(language)
@@ -1635,8 +1635,7 @@ bool Module::FindSourceFile(const FileSpec &orig_spec,
return false;
}
llvm::Optional<std::string>
Module::RemapSourceFile(llvm::StringRef path) const {
std::optional<std::string> Module::RemapSourceFile(llvm::StringRef path) const {
std::lock_guard<std::recursive_mutex> guard(m_mutex);
if (auto remapped = m_source_mappings.RemapPath(path))
return remapped->GetPath();