On Tue, 30 Sep 2025 14:47:03 GMT, Tobias Hartmann <[email protected]> wrote:

>> We hit a guarantee failure because the immediate of the branch instruction 
>> is too large. We should use a `far_jump` instead, like we do for all other 
>> jumps to `StubRoutines::forward_exception_entry()`.
>> 
>> I can reproduce this reliably with 
>> `test/hotspot/jtreg/runtime/valhalla/inlinetypes/classloading/ConcurrentClassLoadingTest.java`
>>  and therefore did not add another regression test.
>> 
>> Thanks,
>> Tobias
>
> Tobias Hartmann has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Increased timeout

Thank you for fixing this, @TobiHartmann. Looks good to me.

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

Marked as reviewed by mhaessig (no project role).

PR Review: 
https://git.openjdk.org/valhalla/pull/1646#pullrequestreview-3285470722

Reply via email to