This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: [PATCH] Fix PR11706, use __builtin_powi[lf] for pow(x, int)
- From: Gabriel Dos Reis <gdr at integrable-solutions dot net>
- To: Richard Guenther <rguenth at tat dot physik dot uni-tuebingen dot de>
- Cc: Paolo Carlini <pcarlini at suse dot de>, gcc-patches at gcc dot gnu dot org, <libstdc++ at gcc dot gnu dot org>
- Date: 07 Feb 2005 16:04:16 +0100
- Subject: Re: [PATCH] Fix PR11706, use __builtin_powi[lf] for pow(x, int)
- Organization: Integrable Solutions
- References: <Pine.LNX.4.44.0502071509070.16464-100000@alwazn.tat.physik.uni-tuebingen.de>
Richard Guenther <rguenth@tat.physik.uni-tuebingen.de> writes:
| On Mon, 7 Feb 2005, Paolo Carlini wrote:
|
| > Richard Guenther wrote:
| >
| > >This patch will fix PR11706 once support for __builtin_powi goes in.
| > >
| > >Bootstrapped and tested on x86_64-unknown-linux-gnu.
| > >
| > >Ok for mainline?
| > >
| > This cannot be ok, because __pow_helper is used also for <complex>...
|
| Indeed. Weird that didn't show up during testing. Grep'ing for
| cmath.tcc also didn't show it because complex doesn't include it.
That is a joke. Look at std_complex.h.
You should not be removing files when you're not faimiliar with the
library structure.
| I'll re-test the following (very) simpler patch then. Ok if
| testing succeeds?
I cannot see that the related has been checked. How do we know that
the PR is indeed fixed with your patch?
-- Gaby