[llvm-commits] [Patch] Add other fp rounding intrinsics to complement llvm.floor

Craig Topper craig.topper at gmail.com
Wed Nov 14 22:51:54 PST 2012


Committed in r168025.

On Wed, Nov 14, 2012 at 9:07 PM, Hal Finkel <hfinkel at anl.gov> wrote:

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



-- 
~Craig
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121114/769d8e31/attachment.html>


More information about the llvm-commits mailing list