[all-commits] [llvm/llvm-project] 202af5: Recommit: [lldb] Remove "dwarf dynamic register si...

Pavel Labath via All-commits all-commits at lists.llvm.org
Thu Oct 7 02:15:20 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 202af507fd1832462ecaf5d9086624bd5f9601bb
      https://github.com/llvm/llvm-project/commit/202af507fd1832462ecaf5d9086624bd5f9601bb
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2021-10-07 (Thu, 07 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/ARC/ABISysV_arc.cpp
    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/Windows/Common/arm/RegisterContextWindows_arm.cpp
    M lldb/source/Plugins/Process/Windows/Common/arm64/RegisterContextWindows_arm64.cpp
    M lldb/source/Plugins/Process/Windows/Common/x64/RegisterContextWindows_x64.cpp
    M lldb/source/Plugins/Process/Windows/Common/x86/RegisterContextWindows_x86.cpp
    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:
  -----------
  Recommit: [lldb] Remove "dwarf dynamic register size expressions" from RegisterInfo

The previous version of the patch did not update the definitions in
conditionally compiled code. This patch includes changes to ARC and
windows targets.

Original commit message was:

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