RFR: 8344104: TestMergeStores fails with ArrayIndexOutOfBoundException [v2]

Dean Long dlong at openjdk.org
Thu Nov 14 23:32:18 UTC 2024


On Thu, 14 Nov 2024 07:05:59 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Test-bug: `RANDOM.nextInt()` would occasionally return a `min_int`. And sadly this overflows: `Math.abs(min_int) == min_int`. Wen we calculate it `% 100`, it still gives us a negative value, and we end up out of bounds. Fixed with a mask.
>
> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
> 
>   do what shipilev said

Marked as reviewed by dlong (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/22080#pullrequestreview-2437392135


More information about the hotspot-compiler-dev mailing list