On Fri, 4 Mar 2022 19:04:40 GMT, Jatin Bhateja <jbhat...@openjdk.org> wrote:
>> IMO RoundTests should have a explicit @run tag without any VM options as >> well. >> >> Do the added VM options run on all platforms in question? What is the >> approximate time to run the test run compared to before? > > Hi @jddarcy , > > Test has been modified on the same lines using generic options which > manipulate compilation thresholds and agnostic to target platforms. > > * @run main/othervm -XX:Tier3CompileThreshold=100 > -XX:CompileThresholdScaling=0.01 -XX:+TieredCompilation RoundTests > > Verified that RoundTests::test* methods gets compiled by c2. > Test execution time with and without change is almost same ~7.80sec over > Skylake-server. > > Regards To be more explicit, the existing RoundTests.java test runs in a fraction of a second. The updated test runs many times slower, even if now under 10 second, at least on some platforms. Can something closer to the original performance be restored? As a tier 1 library test, these tests are run quite frequently. ------------- PR: https://git.openjdk.java.net/jdk/pull/7094
