[all-commits] [llvm/llvm-project] cd05ff: [lldb] Remove distribution_id from ArchSpec

Alex via All-commits all-commits at lists.llvm.org
Wed May 3 10:36:31 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cd05ffdbb2c303cbf476a3a48b3f67094d428e37
      https://github.com/llvm/llvm-project/commit/cd05ffdbb2c303cbf476a3a48b3f67094d428e37
  Author: Alex Langford <alangford at apple.com>
  Date:   2023-05-03 (Wed, 03 May 2023)

  Changed paths:
    M lldb/include/lldb/Host/HostInfoBase.h
    M lldb/include/lldb/Utility/ArchSpec.h
    M lldb/source/Host/linux/HostInfoLinux.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.h
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerCommon.cpp
    M lldb/source/Utility/ArchSpec.cpp

  Log Message:
  -----------
  [lldb] Remove distribution_id from ArchSpec

The qHostInfo packet in the gdb-remote communication protocol specifies
that distribution_id can be set, so lldb handles that. But we store that
in the ArchSpec representing the "Host" platform (whatever platform the
debug server is running on). This field is otherwise unused in ArchSpec,
so it would be a lot easier if we stored that information at the
gdb-remote communication layer.

Sidenote: The distribution_id field is currently unused but I did not
want to remove it in case some folks found it useful (e.g. in downstream
forks).

Differential Revision: https://reviews.llvm.org/D149697




More information about the All-commits mailing list