RFR: 8308966 Add intrinsic for float/double modulo for x86 AVX2 and AVX512 [v2]

[Jatin Bhateja]

On Thu, 1 Jun 2023 16:00:19 GMT, Scott Gibbons <sgibbons at openjdk.org> wrote:

>> Hi @asgibbons ,
>> Kindly also include the results for following benchmark
>> test/micro/org/openjdk/bench/vm/floatingpoint/DremFrem.java
>> 
>> Best Regards,
>> Jatin
>
>> Hi @asgibbons , Kindly also include the results for following benchmark test/micro/org/openjdk/bench/vm/floatingpoint/DremFrem.java
>> 
>> Best Regards, Jatin
> 
> Current top-of-tree results:
> 
> Benchmark                      Mode  Cnt  Score   Error  Units
> DremFrem.calcDoubleJava        avgt   25  7.034 ± 0.001  ns/op
> DremFrem.calcFloatJava         avgt   25  7.011 ± 0.001  ns/op
> DremFrem.cornercaseDoubleJava  avgt   25  5.514 ± 0.006  ns/op
> DremFrem.cornercaseFloatJava   avgt   25  5.510 ± 0.003  ns/op
> 
> 
> My changes:
> 
> Benchmark                      Mode  Cnt  Score   Error  Units
> DremFrem.calcDoubleJava        avgt   25  2.916 ± 0.001  ns/op
> DremFrem.calcFloatJava         avgt   25  4.011 ± 0.001  ns/op
> DremFrem.cornercaseDoubleJava  avgt   25  5.518 ± 0.008  ns/op
> DremFrem.cornercaseFloatJava   avgt   25  5.515 ± 0.007  ns/op

Hi @asgibbons , It will be good to back the special case handlings in the patch with a test case.

Best Regards,
Jatin

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

PR Comment: https://git.openjdk.org/jdk/pull/14224#issuecomment-1575618276