On Wed, 17 Jul 2024 14:30:33 GMT, fabioromano1 <d...@openjdk.org> wrote:
>> I tried on older release, they all agree.
>
> @rgiulietti This is so strange... anyway, I tried also `long x = n * n`,
> `long s = Math.round(Math.sqrt(x >= 0 ? x : x + 0x1p64))` with the test `s <
> n`, which I think it's more mathematically natural, and also this never fails
> for perfect squares.
Also, this avoids a test
if (Long.compareUnsigned(x, s * s - 1) <= 0) { // benign over- and
underflows
s--;
}
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19710#discussion_r1681230093
