RFR: 8339220: [s390x] TestIfMinMax.java failure [v2]

Amit Kumar amitkumar at openjdk.org
Thu Sep 26 11:10:34 UTC 2024


On Thu, 26 Sep 2024 10:06:08 GMT, Amit Kumar <amitkumar at openjdk.org> wrote:

>> This test enables Conditional moves for long operands for s390x. Which fixes the test-case.
>> 
>> Ran tier1 and not saw any regression.
>
> Amit Kumar has updated the pull request incrementally with one additional commit since the last revision:
> 
>   suggestions from Andrew

I did two benchmark, and I see some performance improvement, in `IfMinMax.testSingleLong` and  `IfMinMax.testVectorLong`:


Without Patch: 

Benchmark                   Mode  Cnt      Score      Error  Units
IfMinMax.testReductionInt   avgt   15   2073.648 ±    4.173  ns/op
IfMinMax.testReductionLong  avgt   15   2028.487 ±    1.246  ns/op
IfMinMax.testSingleInt      avgt   15      9.752 ±    0.172  ns/op
IfMinMax.testSingleLong     avgt   15     16.168 ±    0.248  ns/op
IfMinMax.testVectorInt      avgt   15   4713.057 ±   14.566  ns/op
IfMinMax.testVectorLong     avgt   15  27669.122 ± 4096.469  ns/op
Finished running test 'micro:vm.compiler.IfMinMax'


Benchmark                   Mode  Cnt      Score      Error  Units
IfMinMax.testReductionInt   avgt   15   2073.340 ±    4.624  ns/op
IfMinMax.testReductionLong  avgt   15   2028.775 ±    1.874  ns/op
IfMinMax.testSingleInt      avgt   15      9.742 ±    0.172  ns/op
IfMinMax.testSingleLong     avgt   15     16.286 ±    0.177  ns/op
IfMinMax.testVectorInt      avgt   15   4720.292 ±   30.984  ns/op
IfMinMax.testVectorLong     avgt   15  25043.432 ± 1627.543  ns/op
Finished running test 'micro:vm.compiler.IfMinMax'

===================================================================

With Patch: 

Benchmark                   Mode  Cnt     Score    Error  Units
IfMinMax.testReductionInt   avgt   15  2082.858 ± 27.064  ns/op
IfMinMax.testReductionLong  avgt   15  2029.843 ±  4.514  ns/op
IfMinMax.testSingleInt      avgt   15     9.743 ±  0.170  ns/op
IfMinMax.testSingleLong     avgt   15    10.072 ±  0.123  ns/op
IfMinMax.testVectorInt      avgt   15  4775.680 ±  9.953  ns/op
IfMinMax.testVectorLong     avgt   15  4736.881 ± 20.507  ns/op
Finished running test 'micro:vm.compiler.IfMinMax'

Benchmark                   Mode  Cnt     Score    Error  Units
IfMinMax.testReductionInt   avgt   15  2071.544 ±  2.536  ns/op
IfMinMax.testReductionLong  avgt   15  2030.751 ±  3.733  ns/op
IfMinMax.testSingleInt      avgt   15     9.630 ±  0.010  ns/op
IfMinMax.testSingleLong     avgt   15     9.921 ±  0.001  ns/op
IfMinMax.testVectorInt      avgt   15  4774.041 ±  2.562  ns/op
IfMinMax.testVectorLong     avgt   15  4753.247 ± 22.158  ns/op
Finished running test 'micro:vm.compiler.IfMinMax'

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

PR Comment: https://git.openjdk.org/jdk/pull/21198#issuecomment-2376640053


More information about the hotspot-dev mailing list