[Bug middle-end/28252] pow(x,1/3.0) should be converted to cbrt(x)

2006-10-21 Thread uros at gcc dot gnu dot org 


--- Comment #4 from uros at gcc dot gnu dot org  2006-10-21 20:05 ---
Subject: Bug 28252

Author: uros
Date: Sat Oct 21 20:05:35 2006
New Revision: 117937

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=117937
Log:
2006-10-21  Uros Bizjak  [EMAIL PROTECTED]

PR middle-end/28252
* builtins.c (fold_builtin): Fold pow(x,1.0/3.0) as cbrt(x) if
flag_unsafe_math_optimizations is set.

testsuite/ChangeLog:

PR middle-end/28252
* gcc.dg/builtins-8.c: Also check pow(x,1.0/3.0) to cbrt(x)
transformation.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/builtins.c
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gcc.dg/builtins-8.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28252

[Bug middle-end/28252] pow(x,1/3.0) should be converted to cbrt(x)

2006-10-21 Thread pinskia at gcc dot gnu dot org 


--- Comment #5 from pinskia at gcc dot gnu dot org  2006-10-21 21:26 ---
Fixed.


-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.3.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28252

[Bug middle-end/28252] pow(x,1/3.0) should be converted to cbrt(x)

2006-07-06 Thread steven at gcc dot gnu dot org 


--- Comment #3 from steven at gcc dot gnu dot org  2006-07-06 08:57 ---
Uros,

You could post it now, just so that people can have a look at it and maybe
suggest some changes. You know how that goes.

It would be interesting to see if this actually gives speedups...


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28252

[Bug middle-end/28252] pow(x,1/3.0) should be converted to cbrt(x)

2006-07-05 Thread uros at kss-loka dot si 


--- Comment #2 from uros at kss-loka dot si  2006-07-05 08:25 ---
Created an attachment (id=11824)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11824action=view)
Patch to implement pow(x,1.0/3.0) = cbrt(x) optimization

I have the patch that implements the optimization ready, just waiting for the
mainline to open again. Should I post it to gcc-patches anyway?

2006-07-05  Uros Bizjak  [EMAIL PROTECTED]

* builtins.c (fold_builtin): Fold pow(x,1.0/3.0) as cbrt(x) if
flag_unsafe_math_optimizations is set.

testsuite:

* gcc.dg/builtins-8.c: Also check pow(x,1.0/3.0) to cbrt(x)
transformation.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28252

[Bug middle-end/28252] pow(x,1/3.0) should be converted to cbrt(x)

2006-07-05 Thread uros at kss-loka dot si 


-- 

uros at kss-loka dot si changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |uros at kss-loka dot si
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2006-07-04 22:52:33 |2006-07-05 08:26:53
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28252

[Bug middle-end/28252] pow(x,1/3.0) should be converted to cbrt(x)

2006-07-04 Thread rguenth at gcc dot gnu dot org 


--- Comment #1 from rguenth at gcc dot gnu dot org  2006-07-04 22:52 ---
Confirmed.  Whether it's a good idea is another question.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2006-07-04 22:52:33
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28252