On Wed, 28 Jan 2026 18:02:19 GMT, Chris Plummer <[email protected]> wrote:

> Remember states can be stacked, but your implementation only supports one 
> push onto the state stack. I suppose you did this because you never saw a 
> case where the stack was more than one deep.

Yes.
Remember states should be stacked, but I haven't seen that case. So I 
implemented to save (push) once so far.
I can fix it, but I want to do it in subsequent PR (for AArch64) because I will 
update register offset handling for generalization.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29398#discussion_r2739142879

Reply via email to