Committed in r168025.<br><br><div class="gmail_quote">On Wed, Nov 14, 2012 at 9:07 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</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">----- Original Message -----<br>
> From: "Craig Topper" <<a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a>><br>
</div><div class="im">> To: "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>><br>
> Cc: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> Sent: Wednesday, November 14, 2012 10:49:49 PM<br>
> Subject: Re: [llvm-commits] [Patch] Add other fp rounding intrinsics to complement llvm.floor<br>
><br>
> Here's the patch for the language ref part.<br>
<br>
</div>This all looks good (and straightforward) to me.<br>
<br>
Thanks again,<br>
Hal<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
><br>
> On Wed, Nov 14, 2012 at 8:12 PM, Hal Finkel < <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a> ><br>
> wrote:<br>
><br>
><br>
><br>
><br>
> ----- Original Message -----<br>
> > From: "Craig Topper" < <a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a> ><br>
> > To: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> > Sent: Wednesday, November 14, 2012 10:00:59 PM<br>
> > Subject: [llvm-commits] [Patch] Add other fp rounding intrinsics to<br>
> > complement llvm.floor<br>
> ><br>
> ><br>
> > The attached patch adds target independent intrinsics for ceil,<br>
> > trunc, rint, and nearbyint to complement the existing floor<br>
> > intrinsic. This gives direct access to the existing ISD node types.<br>
> > My primary interest is the vector types, as the scalar forms are<br>
> > already accessible through libcall handling. This patch makes<br>
> > scalars available as intrinsics too. Maybe this can simplify<br>
> > libcall<br>
> > handling?<br>
> ><br>
> > I added setOperationAction(Expand) to targets where it was obvious<br>
> > where to do so. Several targets seemed to just have problems<br>
> > returning vector types in general.<br>
><br>
> I assume that we should update the language reference.<br>
><br>
> -Hal<br>
><br>
> ><br>
> > Thanks,<br>
> > ~Craig<br>
> ><br>
> > _______________________________________________<br>
> > llvm-commits mailing list<br>
> > <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
> ><br>
><br>
> --<br>
> Hal Finkel<br>
> Postdoctoral Appointee<br>
> Leadership Computing Facility<br>
> Argonne National Laboratory<br>
><br>
><br>
><br>
> --<br>
> ~Craig<br>
><br>
<br>
--<br>
Hal Finkel<br>
Postdoctoral Appointee<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>