[llvm] r173755 - Teach SDISel to combine fsin / fcos into a fsincos node if the following
Joerg Sonnenberger
joerg at britannica.bec.de
Thu Jan 31 15:00:23 PST 2013
On Thu, Jan 31, 2013 at 04:45:03PM -0600, Hal Finkel wrote:
> ----- Original Message -----
> > From: "Paul Redmond" <paul.redmond at intel.com>
> > To: "Hal Finkel" <hfinkel at anl.gov>
> > Cc: "Evan Cheng" <evan.cheng at apple.com>, "<llvm-commits at cs.uiuc.edu>" <llvm-commits at cs.uiuc.edu>
> > Sent: Thursday, January 31, 2013 4:35:34 PM
> > Subject: Re: [llvm] r173755 - Teach SDISel to combine fsin / fcos into a fsincos node if the following
> >
> > Hi,
> >
> > The attached patch enables this optimization for GNU environments.
> > Please review.
>
> Why are you only making this change for X86? Can we make it more
> generally for GNU environments? glibc is glibc, right? ;)
Well, for x86 in fast-math, it might be useful to do it more generally
as the hardware has a combined instruction.
Joerg
More information about the llvm-commits
mailing list