aprantl created this revision. aprantl added reviewers: JDevlieghere, jingham. aprantl requested review of this revision.
Print the "no plugin" warning only when there is no plugin and not when the typesystem failed to initialize. rdar://72562341 https://reviews.llvm.org/D95992 Files: lldb/source/Target/Process.cpp Index: lldb/source/Target/Process.cpp =================================================================== --- lldb/source/Target/Process.cpp +++ lldb/source/Target/Process.cpp @@ -5684,9 +5684,9 @@ LanguageType language = sc.GetLanguage(); if (language == eLanguageTypeUnknown) return; - auto type_system_or_err = sc.module_sp->GetTypeSystemForLanguage(language); - if (auto err = type_system_or_err.takeError()) { - llvm::consumeError(std::move(err)); + LanguageSet plugins = + PluginManager::GetAllTypeSystemSupportedLanguagesForTypes(); + if (!plugins[language]) { PrintWarning(Process::Warnings::eWarningsUnsupportedLanguage, sc.module_sp.get(), "This version of LLDB has no plugin for the %s language. "
Index: lldb/source/Target/Process.cpp =================================================================== --- lldb/source/Target/Process.cpp +++ lldb/source/Target/Process.cpp @@ -5684,9 +5684,9 @@ LanguageType language = sc.GetLanguage(); if (language == eLanguageTypeUnknown) return; - auto type_system_or_err = sc.module_sp->GetTypeSystemForLanguage(language); - if (auto err = type_system_or_err.takeError()) { - llvm::consumeError(std::move(err)); + LanguageSet plugins = + PluginManager::GetAllTypeSystemSupportedLanguagesForTypes(); + if (!plugins[language]) { PrintWarning(Process::Warnings::eWarningsUnsupportedLanguage, sc.module_sp.get(), "This version of LLDB has no plugin for the %s language. "
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits