Re: [lworld] RFR: 8385328: [lworld] C2: assert(phi->_idx >= nodes_size()) failed: only new Phi per instance memory slice still happens after JDK-8374742 [v8]

[Dean Long]

On Tue, 23 Jun 2026 09:39:51 GMT, Roland Westrelin <rol...@openjdk.org> wrote:

>> I'm OK with this, but I think there should be a comment that explains why it 
>> is safe to do the comparison.  It works because EA would not assign an 
>> instance id if it was possible for it to alias with any other type, right?
>
> I added a comment. Does it look ok to you?

Yes, thanks.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2489#discussion_r3461906178