[all-commits] [llvm/llvm-project] 00e704: [lldb] Remove "dwarf dynamic register size express...

Pavel Labath via All-commits all-commits at lists.llvm.org
Wed Oct 6 04:49:20 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 00e704bf080ffeeb9e334fb3ab71594f9aa50969
      https://github.com/llvm/llvm-project/commit/00e704bf080ffeeb9e334fb3ab71594f9aa50969
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2021-10-06 (Wed, 06 Oct 2021)

  Changed paths:
    M lldb/include/lldb/Target/DynamicRegisterInfo.h
    M lldb/include/lldb/Target/RegisterContext.h
    M lldb/include/lldb/lldb-private-types.h
    M lldb/source/Plugins/ABI/ARM/ABIMacOSX_arm.cpp
    M lldb/source/Plugins/ABI/ARM/ABISysV_arm.cpp
    M lldb/source/Plugins/ABI/Hexagon/ABISysV_hexagon.cpp
    M lldb/source/Plugins/ABI/Mips/ABISysV_mips.cpp
    M lldb/source/Plugins/ABI/Mips/ABISysV_mips64.cpp
    M lldb/source/Plugins/ABI/PowerPC/ABISysV_ppc.cpp
    M lldb/source/Plugins/ABI/SystemZ/ABISysV_s390x.cpp
    M lldb/source/Plugins/Instruction/ARM64/EmulateInstructionARM64.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm64.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextDarwin_i386.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextDarwin_x86_64.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextLinux_i386.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextLinux_x86_64.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextWindows_i386.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextWindows_x86_64.cpp
    M lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_arm.cpp
    M lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_arm64.cpp
    M lldb/source/Plugins/Process/Utility/RegisterInfos_arm.h
    M lldb/source/Plugins/Process/Utility/RegisterInfos_arm64.h
    M lldb/source/Plugins/Process/Utility/RegisterInfos_arm64_sve.h
    M lldb/source/Plugins/Process/Utility/RegisterInfos_i386.h
    M lldb/source/Plugins/Process/Utility/RegisterInfos_mips64.h
    M lldb/source/Plugins/Process/Utility/RegisterInfos_powerpc.h
    M lldb/source/Plugins/Process/Utility/RegisterInfos_ppc64.h
    M lldb/source/Plugins/Process/Utility/RegisterInfos_ppc64le.h
    M lldb/source/Plugins/Process/Utility/RegisterInfos_s390x.h
    M lldb/source/Plugins/Process/Utility/RegisterInfos_x86_64.h
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp
    M lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
    M lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.h
    M lldb/source/Plugins/Process/minidump/RegisterContextMinidump_ARM.cpp
    M lldb/source/Plugins/Process/minidump/RegisterContextMinidump_ARM64.cpp
    M lldb/source/Target/DynamicRegisterInfo.cpp
    M lldb/source/Target/RegisterContext.cpp
    M lldb/unittests/Target/DynamicRegisterInfoTest.cpp
    M lldb/unittests/tools/lldb-server/tests/MessageObjects.cpp

  Log Message:
  -----------
  [lldb] Remove "dwarf dynamic register size expressions" from RegisterInfo

These were added to support some mips registers on linux, but linux mips
support has now been removed due.

They are still referenced in the freebds mips implementation, but the
completeness of that implementation is also unknown. All other
architectures just set these fields to zero, which is a cause of
significant bloat in our register info definitions.

Arm also has registers with variable sizes, but they were implemented in
a more gdb-compatible fashion and don't use this feature.

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




More information about the All-commits mailing list