[all-commits] [llvm/llvm-project] f10e2d: [lldb/Plugins] Have one initializer per ABI plugin

Jonas Devlieghere via All-commits all-commits at lists.llvm.org
Thu Feb 13 21:49:45 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: f10e2df7bc19a3f05038387d77addd16ab058e06
      https://github.com/llvm/llvm-project/commit/f10e2df7bc19a3f05038387d77addd16ab058e06
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2020-02-13 (Thu, 13 Feb 2020)

  Changed paths:
    M lldb/source/API/SystemInitializerFull.cpp
    A lldb/source/Plugins/ABI/AArch64/ABIAArch64.cpp
    A lldb/source/Plugins/ABI/AArch64/ABIAArch64.h
    M lldb/source/Plugins/ABI/AArch64/ABIMacOSX_arm64.cpp
    M lldb/source/Plugins/ABI/AArch64/ABISysV_arm64.cpp
    M lldb/source/Plugins/ABI/AArch64/CMakeLists.txt
    A lldb/source/Plugins/ABI/ARM/ABIARM.cpp
    A lldb/source/Plugins/ABI/ARM/ABIARM.h
    M lldb/source/Plugins/ABI/ARM/CMakeLists.txt
    A lldb/source/Plugins/ABI/Mips/ABIMips.cpp
    A lldb/source/Plugins/ABI/Mips/ABIMips.h
    M lldb/source/Plugins/ABI/Mips/CMakeLists.txt
    A lldb/source/Plugins/ABI/PowerPC/ABIPowerPC.cpp
    A lldb/source/Plugins/ABI/PowerPC/ABIPowerPC.h
    M lldb/source/Plugins/ABI/PowerPC/CMakeLists.txt
    A lldb/source/Plugins/ABI/X86/ABIX86.cpp
    A lldb/source/Plugins/ABI/X86/ABIX86.h
    M lldb/source/Plugins/ABI/X86/CMakeLists.txt
    M lldb/tools/lldb-test/SystemInitializerTest.cpp

  Log Message:
  -----------
  [lldb/Plugins] Have one initializer per ABI plugin

After the recent change that grouped some of the ABI plugins together,
those plugins ended up with multiple initializers per plugin. This is
incompatible with my proposed approach of generating the initializers
dynamically, which is why I've grouped them together in a new entry
point.

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




More information about the All-commits mailing list