LGTM.<br><br>However, I think we can enhance the test cases a little.<br>1. Keep the void* test in builtins-arm.c for apcs-gnu ABI.<br>2. Add the unit test for va_list name mangling.<br><br>Sincerely,<br>Logan<br><br>ps. The attached patch is the revised patch with the updated<br>
test cases.<br><br><div class="gmail_quote">On Thu, Sep 20, 2012 at 6:13 AM, Weiming Zhao <span dir="ltr"><<a href="mailto:weimingz@codeaurora.org" target="_blank">weimingz@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Ping...<br>
<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by<br>
The Linux Foundation<br></blockquote></div>