<div dir="ltr">Thanks for your your review.  Committed as r175954.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Feb 23, 2013 at 7:33 AM, John McCall <span dir="ltr"><<a href="mailto:rjmccall@apple.com" target="_blank">rjmccall@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Feb 22, 2013, at 3:50 AM, Logan Chien <<a href="mailto:tzuhsiang.chien@gmail.com">tzuhsiang.chien@gmail.com</a>> wrote:<br>

>   It seems that both ARMTargetInfo and MipsTargetInfoBase does not override getEHDataRegisterNumber(), thus __builtin_eh_return_data_regno() will always returns -1 on ARM and MIPS.  The patch should fix this issue, and the test cases are included.  Please have a look.  Thanks.<br>

<br>
</div>Looks fine, assuming you have the register numbers right.<br>
<span class="HOEnZb"><font color="#888888"><br>
John.<br>
</font></span></blockquote></div><br></div>