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

Jatin Bhateja jbhateja at openjdk.java.net
Fri Mar 4 19:08:09 UTC 2022


On Fri, 4 Mar 2022 06:06:52 GMT, Joe Darcy <darcy at openjdk.org> wrote:

>> test/jdk/java/lang/Math/RoundTests.java line 32:
>> 
>>> 30:     public static void main(String... args) {
>>> 31:         int failures = 0;
>>> 32:         for (int i = 0; i < 100000; i++) {
>> 
>> Is there an idiom to trigger the auto-vectorization, perhaps using command line arguments, that doesn't bloat the running time of this test?
>
> 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

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

PR: https://git.openjdk.java.net/jdk/pull/7094


More information about the hotspot-compiler-dev mailing list