<div dir="ltr"><div><div><br></div>LGTM.<br><br>Thanks,<br></div>Manman<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 14, 2014 at 11:42 AM, Julien Lerouge <span dir="ltr"><<a href="mailto:jlerouge@apple.com" target="_blank">jlerouge@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=""><br>
<br>
<br>
On Fri, Apr 11, 2014 at 12:41:54PM -0700, Manman Ren wrote:<br>
> Hi Julien,<br>
><br>
> The updated patch looks good overall.<br>
> Since there is refactoring going on, can you separate to two patches, one<br>
> with refactoring but no functionality change, to make review easier?<br>
><br>
> Thanks,<br>
> Manman<br>
><br>
<br>
</div>Hello,<br>
<br>
Attached inline_001.diff that splits the byval argument initialization,<br>
and inline_002.diff that adds the marker for the byval induced allocas<br>
and test case.<br>
<br>
Let me know if there is anything else.<br>
<br>
Thanks again,<br>
Julien<br>
</blockquote></div><br></div>