[Lldb-commits] [PATCH] D96460: [LLDB] Arm64/Linux Add MTE and Pointer Authentication registers

[David Spickett via Phabricator via lldb-commits]

DavidSpickett added a comment.
Herald added a subscriber: JDevlieghere.

@omjavaid I get some new warnings when compiling this on x86:

  [732/1117] Building CXX object too.../RegisterContextDarwin_arm64.cpp.o
  In file included from /work/open_source/lldb-cross-compile/llvm-project/lldb/source/Plugins/Process/Utility/RegisterContextDarwin_arm64.cpp:70:
  /work/open_source/lldb-cross-compile/llvm-project/lldb/source/Plugins/Process/Utility/RegisterInfos_arm64.h:794:35: warning: ‘g_register_infos_mte’ defined but not used [-Wunused-variable]
    794 | static lldb_private::RegisterInfo g_register_infos_mte[] = {
        |                                   ^~~~~~~~~~~~~~~~~~~~
  /work/open_source/lldb-cross-compile/llvm-project/lldb/source/Plugins/Process/Utility/RegisterInfos_arm64.h:791:35: warning: ‘g_register_infos_pauth’ defined but not used [-Wunused-variable]
    791 | static lldb_private::RegisterInfo g_register_infos_pauth[] = {
        |                                   ^~~~~~~~~~~~~~~~~~~~~~

Makes some sense that they're not used but presumably we could mark them as such? In case it's g++ not seeing the existing macros properly, I'm using:

  g++-9 (Ubuntu 9.3.0-23ubuntu1~16.04) 9.3.0


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96460/new/

https://reviews.llvm.org/D96460