<div dir="ltr">Hi Jason,<div><br></div><div>Patch confirmed, plan name changed as expected (and did not change for gcc because gcc describes epilogue as well).</div><div><br></div><div>I made a small change though: now that you brought up "image dump show-unwind", I found that I should make a copy of m_unwind_plan_call_site_sp instead of modifying it directly.</div>
<div>Now "image show-unwind" output looks like:</div><div><br></div><div><div>Asynchronous (not restricted to call-sites) UnwindPlan for 1`f (start addr 0x8048420):</div><div>This UnwindPlan originally sourced from eh_frame CFI plus augmentation from assembly parsing</div>
<div>Address range of this UnwindPlan: [1..text + 256-0x0000014b)</div><div>row[0]: 0x00000000: CFA=esp +4 => eip=[esp] </div><div>row[1]: 0x00000003: CFA=esp+24 => eip=[esp+20] </div><div>row[2]: 0x0000004a: CFA=esp +4 => eip=[esp] </div>
<div><br></div><div>Synchronous (restricted to call-sites) UnwindPlan for 1`f (start addr 0x8048420):</div><div>This UnwindPlan originally sourced from eh_frame CFI</div><div>Address range of this UnwindPlan: [1..text + 256-0x0000014b)</div>
<div>row[0]: 0x00000000: CFA=esp +4 => eip=[esp] </div><div>row[1]: 0x00000003: CFA=esp+24 => eip=[esp+20] </div><div><br></div><div>Architecture default UnwindPlan for 1`f (start addr 0x8048420):</div><div>This UnwindPlan originally sourced from i386 default unwind plan</div>
<div>row[0]: 0x00000000: CFA=ebp +8 => esp=ebp+8 ebp=[ebp] eip=[ebp+4] </div></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 25, 2014 at 2:45 PM, Tong Shen <span dir="ltr"><<a href="mailto:endlessroad@google.com" target="_blank">endlessroad@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Of course :-)<div><div>Keeping the old name may cause confusion in the future.</div><div><br></div><div>
I will test it now and make sure it works.</div></div><div><br></div><div>Thanks!</div></div><div class="gmail_extra"><div class="">
<br><br><div class="gmail_quote">On Mon, Aug 25, 2014 at 2:43 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, I think it might be useful to note that the UnwindPlan has been modified (in case one of us is looking at it with "image dump show-unwind").  What do you think about this patch?  I don't have eh_frame info on my system right now so I haven't tested this.<br>


<br>
</blockquote></div><br><br clear="all"><div><br></div></div><span class="HOEnZb"><font color="#888888">-- <br><div dir="ltr">Best Regards, Tong Shen</div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Best Regards, Tong Shen</div>
</div>