RFR: 8279508: Auto-vectorize Math.round API [v9]
Jatin Bhateja
jbhateja at openjdk.java.net
Fri Mar 11 19:10:11 UTC 2022
On Thu, 10 Mar 2022 14:29:36 GMT, Joe Darcy <darcy at openjdk.org> wrote:
>> 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.
Hi @jddarcy ,
Earlier none of the test methods in RoundTests.java were compiled on account of low invocation count, a loop with 2000 iterations under the influence controlled compilation threshold now triggers tier4 compilation of test points. I did several runs in Skylake machine with patch and without patch and could see no perceptible difference in runtime due to modification.
I have further reduced the invocation count and compile threshold.
Thanks
-------------
PR: https://git.openjdk.java.net/jdk/pull/7094
More information about the core-libs-dev
mailing list