RFR: 8189107 - AARCH64: create intrinsic for pow

Dmitrij Pochepko dmitrij.pochepko at bell-sw.com
Thu Sep 13 14:35:53 UTC 2018


Hi,

I found 3 items to fix in your comments in 
http://cr.openjdk.java.net/~adinn/8189107/original_algorithm.txt


1)

//           [1, sqrt(3/2)), [sqrt(3/2), sqrt(3)/2), [sqrt(3)/2, 2)
//      i.e. [1, ~1.225],    [~1.225,    ~1.732),    [~1.732, 2)

this one should be:

[1, sqrt(3/2)), [sqrt(3/2), sqrt(3)), [sqrt(3), 2)
i.e. [1, ~1.225],    [~1.225,    ~1.732),    [~1.732,    2)


2)

"4) Filter out overflows (z > 1023) or underflows (z < -1077)"

should be:

"4) Filter out overflows (z > 1023) or underflows (z < -1076)"


3) "5) Let |z| = n + r where n is int, 0 <= n < 10, and 0 <= r < 1"

should be:

"5) Let |z| = n + r where n is int, 0 <= n < 1076, and 0 <= r < 1"


Other comments seems fine

Thanks,

Dmitrij


On 07/09/18 15:58, Andrew Dinn wrote:
>
> I have rewritten the algorithm to achieve what I think is needed to
> patch these omissions. The redraft of this part of the code is available
> here:
>
>    http://cr.openjdk.java.net/~adinn/8189107/original_algorithm.txt
>
>



More information about the hotspot-compiler-dev mailing list