On Fri, 18 Apr 2025 18:25:54 GMT, Chen Liang <li...@openjdk.org> wrote:

>> Perf numbers for simple main:
>> Linking of `Class::forName0` down from ~152 to ~83
>> 
>> For calling little color management system 
>> https://bugs.openjdk.org/browse/JDK-8313344:
>> JNI: ~45
>> baseline panama: ~164
>> patch: ~103
>> 
>> Also see #24705.
>
> Chen Liang has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Simplify FFM linker warmup

src/java.base/share/classes/jdk/internal/foreign/abi/NativeEntryPoint.java line 
57:

> 55:             if (!(o instanceof CacheKey other)) return false;
> 56: 
> 57:             return methodType == other.methodType && abi == other.abi && 
> capturedStateMask == other.capturedStateMask

Is this correct? The old code was using `MethodType::equals`, but now we seem 
to use `==` instead. I suppose the idea is that method types are interned, and 
their `equals` method is only really meaningful when checking whether a method 
type with same shape has already been interned?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24742#discussion_r2063500588

Reply via email to