<div dir="ltr">In i386 .eh_frame, esp=4, ebp=5. I confirmed it by cfi directives gcc generates and objdump -W.<div><br></div><div>So it's either we chose the wrong eRegisterKindXXX, or we put {dwarf, gcc}_{esp, ebp}_i386 in wrong order in source/Plugins/Process/Utility/RegisterContext_x86.h :-)</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 12, 2014 at 3:23 PM, Jason Molenda <span dir="ltr"><<a href="mailto:jmolenda@apple.com" target="_blank">jmolenda@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Tong, sorry for not replying earlier.<br>
<br>
I don't understand why the current code is wrong.<br>
<br>
        eRegisterKindGCC = 0,    // the register numbers seen in eh_frame<br>
        eRegisterKindDWARF,      // the register numbers seen DWARF<br>
<br>
and<br>
<br>
        SectionSP sect = sl->FindSectionByType (eSectionTypeEHFrame, true);<br>
        if (sect.get())<br>
        {<br>
            m_eh_frame = new DWARFCallFrameInfo(m_object_file, sect, eRegisterKindGCC, true);<br>
        }<br>
<br>
We create a DWARFCallFrameInfo object to read the eh_frame information and we tell it that the register kind to use for the reg numbers is eRegisterKindGCC - aka the eh_frame register numbering domain.<br>
<br>
This all looks correct to me, and designed specifically to handle the case of that ancient gcc register numbering bug in the eh_frame section for i386.<br>
<br>
We never read the actual DWARF debug_frame (today) because I've never seen a case where it had more information than the eh_frame unwind info.<br>
<div><div class="h5"><br>
<br>
<br>
> On Aug 12, 2014, at 1:36 PM, Tong Shen <<a href="mailto:endlessroad@google.com">endlessroad@google.com</a>> wrote:<br>
><br>
> Aha, got it.<br>
> Patch confirmed and attached.<br>
><br>
><br>
> On Tue, Aug 12, 2014 at 12:51 PM, Ed Maste <<a href="mailto:emaste@freebsd.org">emaste@freebsd.org</a>> wrote:<br>
> On 12 August 2014 15:43, Todd Fiala <<a href="mailto:tfiala@google.com">tfiala@google.com</a>> wrote:<br>
> ><br>
> > Hey Tong,<br>
> ><br>
> > Ah interesting.  The register infos in lldb have a vector of different register numbers (I think we call them "register kinds") based on the type of encoding you're using.  Let's have a look at that to see if we can adjust the code to query for the right register kind.  We'll look at this today!<br>

><br>
> There was a thread on this topic on this list a while back, and I<br>
> think the two register sets in question here are DWARF and GCC, which<br>
> are identical in all cases except that i386 swaps esp and ebp between<br>
> the two.  This is all the legacy of an ancient GCC bug.  So I think<br>
> you're right Todd, just picking the other set should solve this.<br>
><br>
><br>
><br>
> --<br>
> Best Regards, Tong Shen<br>
</div></div>> <fix_register_kind.patch>_______________________________________________<br>
<div class="HOEnZb"><div class="h5">> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Best Regards, Tong Shen</div>
</div>