RFR: 8349764: RISC-V: C1: Improve Class.isInstance intrinsic

[Fei Yang]

On Tue, 11 Feb 2025 03:15:47 GMT, Gui Cao <gcao at openjdk.org> wrote:

> Follow this patch https://github.com/openjdk/jdk/pull/22491, RISC-V implementation for Class.isInstance intrinsic.
> 
> 
> ### JMH numbers (tested on milkv megrez with hotspot client build):
> 
> #### before this patch:
> 
> Benchmark                             Mode  Cnt    Score   Error  Units
> SecondarySupersLookup.testNegative00  avgt   15   48.589 ± 0.981  ns/op
> SecondarySupersLookup.testNegative01  avgt   15   48.577 ± 0.297  ns/op
> SecondarySupersLookup.testNegative02  avgt   15   48.760 ± 0.740  ns/op
> SecondarySupersLookup.testNegative03  avgt   15   48.442 ± 0.029  ns/op
> SecondarySupersLookup.testNegative04  avgt   15   48.453 ± 0.095  ns/op
> SecondarySupersLookup.testNegative05  avgt   15   48.435 ± 0.025  ns/op
> SecondarySupersLookup.testNegative06  avgt   15   48.540 ± 0.476  ns/op
> SecondarySupersLookup.testNegative07  avgt   15   48.452 ± 0.032  ns/op
> SecondarySupersLookup.testNegative08  avgt   15   48.466 ± 0.034  ns/op
> SecondarySupersLookup.testNegative09  avgt   15   48.478 ± 0.132  ns/op
> SecondarySupersLookup.testNegative10  avgt   15   48.435 ± 0.032  ns/op
> SecondarySupersLookup.testNegative16  avgt   15   48.440 ± 0.027  ns/op
> SecondarySupersLookup.testNegative20  avgt   15   47.977 ± 0.989  ns/op
> SecondarySupersLookup.testNegative30  avgt   15   48.655 ± 0.487  ns/op
> SecondarySupersLookup.testNegative32  avgt   15   48.566 ± 0.251  ns/op
> SecondarySupersLookup.testNegative40  avgt   15   48.513 ± 0.196  ns/op
> SecondarySupersLookup.testNegative50  avgt   15   48.454 ± 0.075  ns/op
> SecondarySupersLookup.testNegative55  avgt   15   71.670 ± 1.632  ns/op
> SecondarySupersLookup.testNegative56  avgt   15   70.923 ± 1.679  ns/op
> SecondarySupersLookup.testNegative57  avgt   15   70.140 ± 0.048  ns/op
> SecondarySupersLookup.testNegative58  avgt   15   70.473 ± 0.726  ns/op
> SecondarySupersLookup.testNegative59  avgt   15   70.127 ± 0.022  ns/op
> SecondarySupersLookup.testNegative60  avgt   15   82.525 ± 1.178  ns/op
> SecondarySupersLookup.testNegative61  avgt   15   81.647 ± 0.758  ns/op
> SecondarySupersLookup.testNegative62  avgt   15   82.347 ± 1.943  ns/op
> SecondarySupersLookup.testNegative63  avgt   15  129.188 ± 1.550  ns/op
> SecondarySupersLookup.testNegative64  avgt   15  130.274 ± 1.668  ns/op
> SecondarySupersLookup.testPositive01  avgt   15   63.390 ± 0.222  ns/op
> SecondarySupersLookup.testPositive02  avgt   15   63.435 ± 0.027  ns/op
> SecondarySupersLookup.testPositive03  avgt   15   63.469 ± 0.080  ns/op
> SecondarySupersLookup.testPositive04  avgt   15   63.896 ...

Looks fine to me. Thanks.

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

Marked as reviewed by fyang (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23551#pullrequestreview-2611013226