<p dir="ltr">Thanks your review!</p>
<p dir="ltr">Hi Richard, I guess lib/Sema is owned by you. Could you help commit if ok? Or who should I ask to help review and commit?</p>
<p dir="ltr">Thanks!</p>
<div class="gmail_quote">2013/6/25 下午10:10 於 "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> 寫道：<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
LGTM<br>
<br>
On 24 June 2013 21:35, WenHan Gu (谷汶翰) <<a href="mailto:wenhan.gu@gmail.com">wenhan.gu@gmail.com</a>> wrote:<br>
> Merge into existing testcase. Thanks!<br>
><br>
> 2013/6/25 上午3:34 於 "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> 寫道：<br>
><br>
>> The test fails, since clang with -verify is looking for diagnostics to<br>
>> match. You need to add expected-no-diagnostics to the test or if<br>
>> possible merge the test with an existing file to reduce the number of<br>
>> distinct tests that get run during a check-clang invocation.<br>
>><br>
>> On 24 June 2013 14:34, WenHan Gu (谷汶翰) <<a href="mailto:wenhan.gu@gmail.com">wenhan.gu@gmail.com</a>> wrote:<br>
>> > Attached.<br>
>> > Sorry for inconvenience.<br>
>> ><br>
>> > 2013/6/24 下午10:20 於 "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> 寫道：<br>
>> ><br>
>> >> Your second patch is a link to google docs and it is failing to<br>
>> >> download<br>
>> >> for me. Would you mind just attaching the patch?<br>
>> >><br>
>> >> Thanks,<br>
>> >> Rafael<br>
>> >><br>
>> ><br>
</blockquote></div>