[all-commits] [llvm/llvm-project] 687939: [lldb] Replace Host::SystemLog with Debugger::Repo...

Jonas Devlieghere via All-commits all-commits at lists.llvm.org
Fri Jun 24 09:46:41 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6879391908cab68e8d43a0097fa8c9cd3b86eff9
      https://github.com/llvm/llvm-project/commit/6879391908cab68e8d43a0097fa8c9cd3b86eff9
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2022-06-24 (Fri, 24 Jun 2022)

  Changed paths:
    M lldb/include/lldb/Host/Host.h
    M lldb/source/Core/Module.cpp
    M lldb/source/Host/common/Host.cpp
    M lldb/source/Host/macosx/objcxx/Host.mm
    M lldb/source/Interpreter/Options.cpp
    M lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderDarwin.cpp
    M lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp
    M lldb/source/Symbol/CompactUnwindInfo.cpp
    M lldb/source/Symbol/DWARFCallFrameInfo.cpp
    M lldb/source/Symbol/Function.cpp
    M lldb/source/Symbol/SymbolContext.cpp
    M lldb/test/API/lang/objc/modules-hash-mismatch/TestClangModulesHashMismatch.py
    M lldb/tools/lldb-server/lldb-platform.cpp

  Log Message:
  -----------
  [lldb] Replace Host::SystemLog with Debugger::Report{Error,Warning}

As it exists today, Host::SystemLog is used exclusively for error
reporting. With the introduction of diagnostic events, we have a better
way of reporting those. Instead of printing directly to stderr, these
messages now get printed to the debugger's error stream (when using the
default event handler). Alternatively, if someone is listening for these
events, they can decide how to display them, for example in the context
of an IDE such as Xcode.

This change also means we no longer write these messages to the system
log on Darwin. As far as I know, nobody is relying on this, but I think
this is something we could add to the diagnostic event mechanism.

Differential revision: https://reviews.llvm.org/D128480




More information about the All-commits mailing list