On Wed, 6 May 2026 07:58:44 GMT, Aleksey Shipilev <[email protected]> wrote:

>> Andrew Haley has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains 111 commits:
>> 
>>  - Merge branch 'master' into JDK-8134940
>>  - Review comments
>>  - Review comments
>>  - Add FrameMap::last_fpu_reg()
>>  - Review comments
>>  - Review comments
>>  - Review comments
>>  - Merge branch 'JDK-8134940' of https://github.com/theRealAph/jdk into 
>> JDK-8134940
>>  - More
>>  - Merge branch 'JDK-8134940' of https://github.com/theRealAph/jdk into 
>> JDK-8134940
>>  - ... and 101 more: https://git.openjdk.org/jdk/compare/3e1d0b8e...4610777b
>
> src/hotspot/share/runtime/sharedRuntime.cpp line 2585:
> 
>> 2583:     return;
>> 2584:   }
>> 2585: 
> 
> This looks unrelated to PR, and at least indenting is broken. Maybe you want 
> to pull it out as separate change?

Mm, yes. It's an obvious bug,
Maybe we need a blanket PR for all these out-of-memory bugs during JVM boostrap.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28541#discussion_r3357418231

Reply via email to