[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code
From: |
rguenth at tat dot physik dot uni-tuebingen dot de |
Subject: |
[Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code |
Date: |
11 Mar 2004 20:14:16 -0000 |
------- Additional Comments From rguenth at tat dot physik dot uni-tuebingen
dot de 2004-03-11 20:14 -------
Subject: Re: std::pow(T, int) implementation pessimizes
code
gdr at integrable-solutions dot net wrote:
> ------- Additional Comments From gdr at integrable-solutions dot net
> 2004-03-10 20:43 -------
> Subject: Re: std::pow(T, int) implementation pessimizes code
>
> "pcarlini at suse dot de" <address@hidden> writes:
>
> | ------- Additional Comments From pcarlini at suse dot de 2004-03-10 20:26
> -------
> | Subject: Re: std::pow(T, int) implementation pessimizes
> | code
> |
> | gdr at integrable-solutions dot net wrote:
> |
> | >Did you get the report that ::pow() is no longer inlined?
> | >
> | >
> | Why, now you are discussing ::pow, if in your comment #6 and #8 you
> | emphasized that we should not use ::pow?
>
> I'm discussing ::pow() because that is what the reporter wanted V3 to
> use. I emphasize against those, and I would strongly advise against
> any patch that does that. Which is why I'm telling you that upfront
> before you waste your time in that direction.
> The only fix that I should have applied is to make __cmath_power()
> inline. Any other if or #if are just papering over the fundamental
> problem. That is not fix.
As I said numerous times, inlining __cmath_power() will make things
worse, because you need a _very_ clever inliner to notice the profitable
cases. My preffered solution would be a added __builtin_cmath_power()
with appropriate folders.
Richard.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11706
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, (continued)
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, pcarlini at suse dot de, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, gdr at integrable-solutions dot net, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, pcarlini at suse dot de, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, gdr at integrable-solutions dot net, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, pcarlini at suse dot de, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, gdr at integrable-solutions dot net, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, pcarlini at suse dot de, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, bangerth at dealii dot org, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, pcarlini at suse dot de, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, gdr at integrable-solutions dot net, 2004/03/11
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code,
rguenth at tat dot physik dot uni-tuebingen dot de <=
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, rguenth at tat dot physik dot uni-tuebingen dot de, 2004/03/12
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, rguenth at tat dot physik dot uni-tuebingen dot de, 2004/03/12
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, rakdver at gcc dot gnu dot org, 2004/03/13
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, pcarlini at suse dot de, 2004/03/13
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, cvs-commit at gcc dot gnu dot org, 2004/03/13
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, cvs-commit at gcc dot gnu dot org, 2004/03/13
- [Bug libstdc++/11706] std::pow(T, int) implementation pessimizes code, gdr at integrable-solutions dot net, 2004/03/13