RFR: 8354996: Reduce dynamic code generation for a single downcall [v2]

Chen Liang liach at openjdk.org
Tue Apr 22 15:00:41 UTC 2025


On Tue, 22 Apr 2025 08:03:30 GMT, Viktor Klang <vklang at openjdk.org> wrote:

>> 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/LinkerOptions.java line 149:
> 
>> 147:         @Override
>> 148:         public boolean equals(Object obj) {
>> 149:             return obj instanceof FirstVariadicArg that && index == that.index;
> 
> Might be worth checking `obj == this || obj instanceof ...`

I think this is still unlikely to hit (it is possible if people intentionally reuse the value of a `Linker.Option.firstVariadicArg` call), most of the usages will see non-identity equality comparisons.

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

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


More information about the core-libs-dev mailing list