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 Still good! ------------- Marked as reviewed by chagedorn (Committer). PR Review: https://git.openjdk.org/valhalla/pull/2306#pullrequestreview-4073218066
