[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