RFR: 8279508: Auto-vectorize Math.round API [v9]

[Joe Darcy]

On Fri, 4 Mar 2022 19:04:40 GMT, Jatin Bhateja <jbhateja at 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