RFR: 8349686: [s390x] C1: Improve Class.isInstance intrinsic [v7]

[Andrew Haley]

On Fri, 14 Feb 2025 14:53:55 GMT, Amit Kumar <amitkumar at openjdk.org> wrote:

>> Amit Kumar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   space for 3 registers
>
> New benchmark result: 
> 
> 
> Benchmark                             Mode  Cnt   Score   Error  Units
> SecondarySupersLookup.testNegative00  avgt   15   4.271 ± 0.034  ns/op
> SecondarySupersLookup.testNegative01  avgt   15   4.270 ± 0.048  ns/op
> SecondarySupersLookup.testNegative02  avgt   15   4.263 ± 0.019  ns/op
> SecondarySupersLookup.testNegative03  avgt   15   4.266 ± 0.023  ns/op
> SecondarySupersLookup.testNegative04  avgt   15   4.274 ± 0.030  ns/op
> SecondarySupersLookup.testNegative05  avgt   15   4.268 ± 0.019  ns/op
> SecondarySupersLookup.testNegative06  avgt   15   4.269 ± 0.022  ns/op
> SecondarySupersLookup.testNegative07  avgt   15   4.280 ± 0.027  ns/op
> SecondarySupersLookup.testNegative08  avgt   15   4.274 ± 0.030  ns/op
> SecondarySupersLookup.testNegative09  avgt   15   4.258 ± 0.012  ns/op
> SecondarySupersLookup.testNegative10  avgt   15   4.266 ± 0.023  ns/op
> SecondarySupersLookup.testNegative16  avgt   15   4.257 ± 0.010  ns/op
> SecondarySupersLookup.testNegative20  avgt   15   4.258 ± 0.011  ns/op
> SecondarySupersLookup.testNegative30  avgt   15   4.260 ± 0.019  ns/op
> SecondarySupersLookup.testNegative32  avgt   15   4.263 ± 0.024  ns/op
> SecondarySupersLookup.testNegative40  avgt   15   4.260 ± 0.013  ns/op
> SecondarySupersLookup.testNegative50  avgt   15   4.266 ± 0.024  ns/op
> SecondarySupersLookup.testNegative55  avgt   15  28.628 ± 2.120  ns/op
> SecondarySupersLookup.testNegative56  avgt   15  28.561 ± 0.477  ns/op
> SecondarySupersLookup.testNegative57  avgt   15  30.626 ± 3.137  ns/op
> SecondarySupersLookup.testNegative58  avgt   15  29.328 ± 0.528  ns/op
> SecondarySupersLookup.testNegative59  avgt   15  32.580 ± 4.115  ns/op
> SecondarySupersLookup.testNegative60  avgt   15  32.745 ± 3.782  ns/op
> SecondarySupersLookup.testNegative61  avgt   15  33.227 ± 3.922  ns/op
> SecondarySupersLookup.testNegative62  avgt   15  33.354 ± 3.655  ns/op
> SecondarySupersLookup.testNegative63  avgt   15  35.595 ± 3.865  ns/op
> SecondarySupersLookup.testNegative64  avgt   15  34.268 ± 3.374  ns/op
> SecondarySupersLookup.testPositive01  avgt   15   4.800 ± 0.010  ns/op
> SecondarySupersLookup.testPositive02  avgt   15   4.803 ± 0.017  ns/op
> SecondarySupersLookup.testPositive03  avgt   15   4.799 ± 0.012  ns/op
> SecondarySupersLookup.testPositive04  avgt   15   4.799 ± 0.012  ns/op
> SecondarySupersLookup.testPositive05  avgt   15   4.797 ± 0.007  ns/op
> SecondarySupersLookup.testPositive06  avgt   15   4.798 ± 0.013  ns/op
> SecondarySupersLookup.testPositive07  avgt   15   4.803 ± 0.0...

> @offamitkumar See #23572 I've deleted the isInstance native call so it's no longer an intrinsic.

I see only Class.isArray(), Class.isInterface() and Class.isPrimitive(). Class.isInstance() seems to be untouched?

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

PR Comment: https://git.openjdk.org/jdk/pull/23535#issuecomment-2672177877