RFR: 8282661: [BACKOUT] ByteBufferTest.java: replace endless recursion with RuntimeException in void ck(double x, double y) [v2]

[Quan Anh Mai]

On Fri, 4 Mar 2022 15:50:45 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Revert "8282573: ByteBufferTest.java: replace endless recursion with RuntimeException in void ck(double x, double y)"
>>    
>>    This reverts commit a584c904a9e386d7ce80fb9cc6d49fece065d3da.
>>  - Revert "8282661: 2 compiler/intrinsics/unsafe/*ByteBufferTest.java tests fail after JDK-8282573"
>>    
>>    This reverts commit 684e7b3f55d8f109eb096b1294600fa9fa8fbfbd.
>
> test/hotspot/jtreg/compiler/intrinsics/unsafe/ByteBufferTest.java line 219:
> 
>> 217: 
>> 218:     void ck(double x, double y) {
>> 219:         if (x == x && y == y && x != y) {
> 
> This line of logic is begging for a comment of some sort.
> Especially to keep someone from coming along later and "optimizing" it.

I suggest using `Double::isNan` instead of `x == x`

-------------

PR: https://git.openjdk.java.net/jdk/pull/7699