<p dir="ltr">Unfortunately I can't. The unwind code depends on the 32-on-64 code in order to give a valid back trace so becomes pointless without it. I guess I could separate them and there would be no regression from the current state, there just would be no gain.</p>
<div class="gmail_quote">On Nov 9, 2014 1:01 PM, "Ed Maste" <<a href="mailto:emaste@freebsd.org">emaste@freebsd.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'd like to think about the 32-on-64 implementation some more. Can you submit the unwind changes separately first?<br>
<br>
================<br>
Comment at: source/Plugins/ABI/SysV-ppc/ABISysV_ppc.cpp:1015<br>
@@ -1014,4 +1014,3 @@<br>
<br>
-    const int32_t ptr_size = 8;<br>
-    row->SetCFARegister (LLDB_REGNUM_GENERIC_SP);<br>
-    row->SetCFAOffset (8);<br>
+    const int32_t ptr_size = 4;<br>
+    row->SetCFARegister (sp_reg_num);<br>
----------------<br>
I'd specifically call out this fix in the commit message<br>
<br>
<a href="http://reviews.llvm.org/D6183" target="_blank">http://reviews.llvm.org/D6183</a><br>
<br>
<br>
</blockquote></div>