On Fri, Oct 5, 2012 at 2:25 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank" class="cremed">hfinkel@anl.gov</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
<br>
----- Original Message -----<br>
> From: "William J. Schmidt" <<a href="mailto:wschmidt@linux.vnet.ibm.com" class="cremed">wschmidt@linux.vnet.ibm.com</a>><br>
> To: <a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
> Sent: Friday, October 5, 2012 4:20:53 PM<br>
> Subject: [llvm-commits] [PATCH, PowerPC] Split Darwin and 64-bit SVR4 processing of LowerFormalArguments<br>
><br>
> The attached patch is some TLC to split apart a function being used<br>
> by<br>
> both Darwin and 64-bit SVR4 ABIs.  There is no change in function.  I<br>
> have plans to make some nontrivial changes to the SVR4 ABI handling,<br>
> and<br>
> having the two ABIs separated will be better for all concerned.<br>
><br>
> Tested with no new regressions.  Ok to commit?<br>
<br>
</div>Alright (I assume the large block is copied from the Darwin version), LGTM.<br></blockquote><div><br></div><div>Maybe I'm just confused, but why is it OK to copy/paste so much code from Darwin? IS the code going away as you refactor it, or is there actually a need to build an abstraction shared between the two?</div>
<div><br></div><div>(I may just not understand the end goal of the patch, in which case a better explanation will fix matters...)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
 -Hal<br>
<div class="im"><br>
><br>
> Thanks,<br>
> Bill<br>
> --<br>
> Bill Schmidt, Ph.D.<br>
> IBM Advance Toolchain for PowerLinux<br>
> IBM Linux Technology Center<br>
> <a href="mailto:wschmidt@us.ibm.com" class="cremed">wschmidt@us.ibm.com</a><br>
> <a href="mailto:wschmidt@linux.vnet.ibm.com" class="cremed">wschmidt@linux.vnet.ibm.com</a><br>
><br>
><br>
><br>
><br>
><br>
><br>
</div>> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Hal Finkel<br>
Postdoctoral Appointee<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" class="cremed">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</font></span></blockquote></div><br></div>