On Mon, 18 May 2026 19:58:11 GMT, Tobias Hartmann <[email protected]> wrote:

>> This is an improved fix for an earlier incomplete fix (#1716). The problem 
>> with the original fix is that I assumed that the branch with the stress 
>> counter only ever contains the stress counter. The failure in the bug 
>> description shows that this assumption is incorrect.
>> 
>> The fix is to wire the full memory state instead. For completeness, I also 
>> wired the I/O state, just to be sure that this fix will not also be 
>> incomplete.
>> 
>> Testing:
>>  - [x] tier1,tier2,tier3 plus addtional stress testing with 
>> `--enable-preview`
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> That looks good to me.

Thank you for the review, @TobiHartmann.

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

PR Comment: https://git.openjdk.org/valhalla/pull/2448#issuecomment-4485175258

Reply via email to