On Mon, 18 May 2026 14:40:49 GMT, Manuel Hässig <[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).

This pull request has now been integrated.

Changeset: af9a0238
Author:    Manuel Hässig <[email protected]>
Committer: Tobias Hartmann <[email protected]>
URL:       
https://git.openjdk.org/valhalla/commit/af9a02383c71c62aae7410b82d69782a24774cd9
Stats:     23 lines in 2 files changed: 10 ins; 6 del; 7 mod

8383996: [lworld] C2: assert(false) failed: no node with a side effect

Reviewed-by: thartmann

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

PR: https://git.openjdk.org/valhalla/pull/2448

Reply via email to