`test9()` in `TestLWorld.java` is crashing intermittently when running with `StressLoopPeeling`.
The problem can be traced back to a node that is not re-added to the IGVN worklist in `PhiNode::Ideal()`: <img width="282" height="198" alt="graph" src="https://github.com/user-attachments/assets/e8597cf1-d49f-4c8d-b248-184b25f30fba"; /> We are processing `3786 Phi` and update its input `3748 Phi` to another node. As a result, `3748 Phi` loses it's last output and is now dead. It should be re-added to the IGVN worklist to be cleaned up. But that does not happen because we use `set_req()` on `3786 Phi` instead of `set_req_X()` when replacing the input `3748 Phi`. As a result, the dead node stays in the graph and eventually triggers the "no dead use" assert. The fix is straight forward to use `set_req_X()` which ensures that a now dead input node is pushed to the IGVN worklist again and cleaned up later. The affected code is also wrong in mainline but has never triggered there. Therefore, I suggest to wait and not eagerly upstream it to mainline for now. Thanks, Christian ------------- Commit messages: - 8368782: [lworld] C2: fatal error: no reachable node should have no use Changes: https://git.openjdk.org/valhalla/pull/1662/files Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1662&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8368782 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/valhalla/pull/1662.diff Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1662/head:pull/1662 PR: https://git.openjdk.org/valhalla/pull/1662
