Integrated: 8282573: ByteBufferTest.java: replace endless recursion with RuntimeException in void ck(double x, double y)

[Emanuel Peter]

On Thu, 3 Mar 2022 11:24:04 GMT, Emanuel Peter <duke at openjdk.java.net> wrote:

> There are two functions that check for equality.
> void ck(long x, long y): already throws a RuntimeException if we observe inequality
> void ck(double x, double y): called itself, leading to endless recursion and a StackOverflowError.
> 
> Instead of the recursion, I now also throw a RuntimeException.
> 
> This can currently be triggered with
> `path-to-jtreg/jtreg -va -s -jdk:jdk-path -javaoptions:"-XX:+UnlockDiagnosticVMOptions -XX:-TieredCompilation -XX:+StressGCM -XX:+OptoScheduling -XX:StressSeed=293843391" test/hotspot/jtreg/compiler/intrinsics/unsafe/HeapByteBufferTest.java `
> (see bug [JDK-8282555](https://bugs.openjdk.java.net/browse/JDK-8282555))
> 
> This way I could verify that instead of StackOverflowError we now get RuntimeException, as desired.
> Ran some basic tests to ensure I didn't break things.

This pull request has now been integrated.

Changeset: a584c904
Author:    Emanuel Peter <emanuel.peter at oracle.com>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/a584c904a9e386d7ce80fb9cc6d49fece065d3da
Stats:     4 lines in 1 file changed: 2 ins; 0 del; 2 mod

8282573: ByteBufferTest.java: replace endless recursion with RuntimeException in void ck(double x, double y)

Reviewed-by: psandoz, thartmann

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

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