[lworld] RFR: 8376702: [lworld] MemNode::optimize_simple_memory_chain asserts with "invalid node class: Con" [v2]

[Quan Anh Mai]

On Tue, 3 Feb 2026 09:27:31 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:

>> Hi,
>> 
>> I cannot manage to reproduce the issue, and it seems to fail only once in our CI. From the log, it seems that the issue is due to the fact that the pointer input to the `Load` is a `Proj` but its input dies, so the cast `as_Multi` in `try_optimize_strict_final_load_memory` fails. I can see a similar check in `MemNode::optimize_simple_memory_chain`. As a result, it is reasonable to just return the top here.
>> 
>> Please take a look and leave your review, thanks a lot.
>
> Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Enable tests from 8325632

There seems to be a bunch of timeouts when enabling the tests in `TestNullableInlineTypes`, which is pretty understandable given the enabled tests run 1000 iterations each time they are invoked. I think I will keep them disabled and only enable the test in `TestNullableArrays` first.

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

PR Comment: https://git.openjdk.org/valhalla/pull/2016#issuecomment-3840679449