On Tue, 3 Feb 2026 09:27:31 GMT, Quan Anh Mai <[email protected]> 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

Reply via email to