[Lldb-commits] [PATCH] D49750: Add support for ARM and ARM64 breakpad generated minidump files.

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Jul 31 13:29:42 PDT 2018

clayborg added inline comments.

Comment at: source/Plugins/Process/minidump/ProcessMinidump.cpp:15
 // Other libraries and framework includes
+//#include "lldb/Core/Architecture.h"
 #include "lldb/Core/Module.h"
lemo wrote:
> it this set for removal?
Ah yes!

Comment at: source/Plugins/Process/minidump/RegisterContextMinidump_ARM.cpp:196
+size_t RegisterContextMinidump_ARM::GetRegisterSetCount()  {
+  return k_num_reg_sets;
Not sure why. I was using the latest Xcode and got the warning. RegisterInfo is just a plain struct so it shouldn't cause any issues.


More information about the lldb-commits mailing list