[Lldb-commits] [PATCH] LLDB ARM Register context support

[Tamas Berghammer]

================
Comment at: Plugins/Process/Utility/RegisterInfos_arm.h:185
@@ +184,3 @@
+{   "r6",       NULL,   4,  GPR_OFFSET(6),      eEncodingUint,  eFormatHex,     { gcc_r6,               dwarf_r6,           LLDB_INVALID_REGNUM,        gdb_arm_r6,             gpr_r6      },      NULL,              NULL},
+{   "r7",       NULL,   4,  GPR_OFFSET(7),      eEncodingUint,  eFormatHex,     { gcc_r7,               dwarf_r7,           LLDB_REGNUM_GENERIC_FP,     gdb_arm_r7,             gpr_r7      },      NULL,              NULL},
+{   "r8",       NULL,   4,  GPR_OFFSET(8),      eEncodingUint,  eFormatHex,     { gcc_r8,               dwarf_r8,           LLDB_INVALID_REGNUM,        gdb_arm_r8,             gpr_r8      },      NULL,              NULL},
----------------
Are you sure r7 is FP? I haven't found it in the spec but based on some testing I think it is r11 (on a Nexus 5). Please also add an alias for the fp register the same way it is done for sp.

http://reviews.llvm.org/D8719

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/