[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: signed overflow in atan2
On Wed, Mar 07, 2018 at 05:56:21PM +0700, Robert Elz wrote:
> Date: Wed, 7 Mar 2018 00:33:04 -0800
> From: Eitan Adler <lists%eitanadler.com@localhost>
> Message-ID: <CAF6rxgkPtZu_+m2howVQB5WxdafdrdL2yQJ0VCy2LOVsWT6uNg%mail.gmail.com@localhost>
> | I'd like to commit the patch below. Does anyone have concerns with it?
> The change looks fine technically, but it would be good to see some benchmark
> results before committing it - particularly for the more common case where
> x != 1.0 (but including where x == 2.0 or 0.5) - the change swiches from using
> arithmetic and a single branch to multiple branches (or would with simplisitc
> compilation) and branches are slow wrt prefetch and parallel execution.
I don't see the point other than submitting compiler bug reports. This
is a transformation typically done already.
Main Index |
Thread Index |