This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH]: Add builtin cbrt transformations
- From: Roger Sayle <roger at eyesopen dot com>
- To: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 9 Mar 2005 10:00:25 -0700 (MST)
- Subject: Re: [PATCH]: Add builtin cbrt transformations
On Wed, 9 Mar 2005, Kaveh R. Ghazi wrote:
> 2005-03-09 Kaveh R. Ghazi <ghazi@caip.rutgers.edu>
>
> * builtins.c (fold_builtin_cbrt, fold_builtin_pow): Rearrange
> code. Add more cbrt transformations.
>
> testsuite:
> * gcc.dg/builtins-10.c: Reactivate disabled test.
> * gcc.dg/torture/builtin-power-1.c: Likewise. Also add tests for
> new cbrt transformations.
This is OK for mainline.
I agree that we need to test both flag_unsafe_math_optimizations and
tree_expr_nonnegative_p to avoid problematic cases.
Thanks,
Roger
--