Re: [Fwd: Re: [PATCH] Add powi-to-multiply expansion to cse_sincos pass]

2011-05-24 Thread Richard Guenther
On Tue, May 24, 2011 at 5:28 PM, William J. Schmidt wrote: > > On Tue, 2011-05-24 at 16:44 +0200, Richard Guenther wrote: >> On Tue, May 24, 2011 at 3:38 PM, William J. Schmidt >> wrote: >> > >> > On Tue, 2011-05-24 at 15:11 +0200, Richard Guenther wrote: >> >> On Tue, May 24, 2011 at 2:26 PM, Wi

Re: [Fwd: Re: [PATCH] Add powi-to-multiply expansion to cse_sincos pass]

2011-05-24 Thread William J. Schmidt
On Tue, 2011-05-24 at 16:44 +0200, Richard Guenther wrote: > On Tue, May 24, 2011 at 3:38 PM, William J. Schmidt > wrote: > > > > On Tue, 2011-05-24 at 15:11 +0200, Richard Guenther wrote: > >> On Tue, May 24, 2011 at 2:26 PM, William J. Schmidt > >> wrote: > >> > Sure, I'll give that a try this

Re: [PATCH] Add powi-to-multiply expansion to cse_sincos pass

2011-05-24 Thread William J. Schmidt
On Tue, 2011-05-24 at 14:26 +0200, Richard Guenther wrote: > > +/* Recursive subroutine of powi_as_mults. This function takes the > > + array, CACHE, of already calculated exponents and an exponent N and > > + returns a tree that corresponds to CACHE[1]**N, with type TYPE. */ > > + > > +s

Re: [PATCH] Add powi-to-multiply expansion to cse_sincos pass

2011-05-24 Thread Richard Guenther
On Mon, May 23, 2011 at 8:06 PM, William J. Schmidt wrote: > Richard, thanks for the excellent comments.  I really appreciate your > help.  I've implemented them all, and bootstrap/regtest succeeds on > powerpc target.  Here's the revised patch for your consideration. > > Thanks, > Bill > > > 2011

Re: [Fwd: Re: [PATCH] Add powi-to-multiply expansion to cse_sincos pass]

2011-05-24 Thread Richard Guenther
; From: William J. Schmidt >> To: Richard Guenther >> Cc: gcc-patches@gcc.gnu.org >> Subject: Re: [PATCH] Add powi-to-multiply expansion to cse_sincos pass >> Date: Mon, 23 May 2011 13:06:31 -0500 >> >> Richard, thanks for the excellent comments.  I

[Fwd: Re: [PATCH] Add powi-to-multiply expansion to cse_sincos pass]

2011-05-23 Thread William J. Schmidt
change it to a # VUSE <.MEM_19>, in this case? Any pointers to code for similar situations would be appreciated. Thanks, Bill Forwarded Message > From: William J. Schmidt > To: Richard Guenther > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] Add powi-to-mult

Re: [PATCH] Add powi-to-multiply expansion to cse_sincos pass

2011-05-23 Thread William J. Schmidt
Richard, thanks for the excellent comments. I really appreciate your help. I've implemented them all, and bootstrap/regtest succeeds on powerpc target. Here's the revised patch for your consideration. Thanks, Bill 2011-05-23 Bill Schmidt * tree-ssa-math-opts.c (powi_table): New.

Re: [PATCH] Add powi-to-multiply expansion to cse_sincos pass

2011-05-23 Thread Richard Guenther
On Thu, May 19, 2011 at 4:04 PM, William J. Schmidt wrote: > As Richard Guenther suggested, I'm submitting a separate patch to add > powi expansion to multiplies during the cse_sincos pass.  This was > originally submitted as part of a larger patch to fix PR46728. > > Richard noted that tree_expan

[PATCH] Add powi-to-multiply expansion to cse_sincos pass

2011-05-19 Thread William J. Schmidt
As Richard Guenther suggested, I'm submitting a separate patch to add powi expansion to multiplies during the cse_sincos pass. This was originally submitted as part of a larger patch to fix PR46728. Richard noted that tree_expand_builtin_powi, powi_as_mults, and powi_as_mults_1 more properly belo