Did you see my comments on the first round about how the CMake build didn’t work?  Because I don’t see any changes to CMakeLists.txt here, which means it still won’t work.<br><br>The easiest way to make sure you get all the fixes that may have gone in after your initial commit is to revert the revert and build your modifications on top of that.  Then, for example, the fixes to CMakeLists.txt would already be here.<br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 6, 2018 at 11:14 AM Greg Clayton via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg created this revision.<br>
clayborg added reviewers: labath, zturner, markmentovai, javed.absar.<br>
Herald added subscribers: chrib, kristof.beyls.<br>
<br>
In this patch I add support for ARM and ARM64 break pad files. There are two flavors of ARM: Apple where FP is <a href="https://reviews.llvm.org/source/openmp/" rel="noreferrer" target="_blank">https://reviews.llvm.org/source/openmp/</a>, and non Apple where FP is <a href="https://reviews.llvm.org/source/libunwind/" rel="noreferrer" target="_blank">https://reviews.llvm.org/source/libunwind/</a>. Added minimal tests that load up ARM64 and the two flavors or ARM core files with a single thread and known register values in each register. Each register is checked for the exact value.<br>
<br>
This is a fixed version of: <a href="https://reviews.llvm.org/D49750" rel="noreferrer" target="_blank">https://reviews.llvm.org/D49750</a><br>
<br>
The changes from <a href="https://reviews.llvm.org/D49750" rel="noreferrer" target="_blank">https://reviews.llvm.org/D49750</a> are:<br>
<br>
- Don't init the m_arch in the Initialize call as a system info isn't required. This keeps the thread list, module list and other tests from failing<br>
- Added -Wextended-offsetof to Xcode project so we catch use extended usages of offsetof before submission<br>
- Fixed any extended offset of warnings<br>
<br>
<br>
<a href="https://reviews.llvm.org/D50336" rel="noreferrer" target="_blank">https://reviews.llvm.org/D50336</a><br>
<br>
Files:<br>
  include/lldb/Target/Target.h<br>
  lldb.xcodeproj/project.pbxproj<br>
  packages/Python/lldbsuite/test/functionalities/postmortem/minidump-new/TestMiniDumpNew.py<br>
  packages/Python/lldbsuite/test/functionalities/postmortem/minidump-new/arm-linux.dmp<br>
  packages/Python/lldbsuite/test/functionalities/postmortem/minidump-new/arm-macos.dmp<br>
  packages/Python/lldbsuite/test/functionalities/postmortem/minidump-new/arm64-macos.dmp<br>
  source/Plugins/Process/minidump/MinidumpParser.cpp<br>
  source/Plugins/Process/minidump/MinidumpParser.h<br>
  source/Plugins/Process/minidump/ProcessMinidump.cpp<br>
  source/Plugins/Process/minidump/RegisterContextMinidump_ARM.cpp<br>
  source/Plugins/Process/minidump/RegisterContextMinidump_ARM.h<br>
  source/Plugins/Process/minidump/RegisterContextMinidump_ARM64.cpp<br>
  source/Plugins/Process/minidump/RegisterContextMinidump_ARM64.h<br>
  source/Plugins/Process/minidump/ThreadMinidump.cpp<br>
  source/Target/Target.cpp<br>
<br>
</blockquote></div>