On Wed, 8 Apr 2026 05:30:56 GMT, Tobias Hartmann <[email protected]> wrote:

>> The test randomly excludes methods from C1/C2 compilation to stress test the 
>> calling convention. While this has been proven to be extremely helpful for 
>> testing the Virtual Threads implementation, it's also extremely slow - 
>> especially when we are unlucky enough for multiple methods to be excluded 
>> both from C1 and C2 compilation. On slow machines and in combination with 
>> other VM flags, test execution time can easily exceed an hour.
>> 
>> While we could now try to improve how these methods are selected, I don't 
>> think it's worth it and we risk wasting significant machine hours just 
>> running methods in the interpreter. I therefore disabled this test feature 
>> by default. It can still be re-enabled for debugging / stress testing 
>> purposes on demand. Long-term, we should add this capability to the VM. I 
>> filed [JDK-8381751](https://bugs.openjdk.org/browse/JDK-8381751) for this.
>> 
>> Thanks,
>> Tobias
>
> Tobias Hartmann has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Lower iteration count, moved to tier2

Thanks again Christian!

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

PR Comment: https://git.openjdk.org/valhalla/pull/2306#issuecomment-4204490185

Reply via email to