RFR: 8317545: AIX PPC64: Implementation of Foreign Function & Memory API [v2]

Martin Doerr mdoerr at openjdk.org
Fri Oct 13 12:26:48 UTC 2023


On Fri, 13 Oct 2023 10:21:06 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> Martin Doerr has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Improve micro benchmarks and a comment.
>
> test/micro/org/openjdk/bench/java/lang/foreign/CLayouts.java line 68:
> 
>> 66:      * The layout for the {@code double} C type
>> 67:      */
>> 68:     public static final ValueLayout.OfDouble C_DOUBLE = ValueLayout.JAVA_DOUBLE.withByteAlignment(IS_AIX ? 4 : 8);
> 
> Would prefer if this used `Linker.canonicalLayouts`
> Suggestion:
> 
>     public static final ValueLayout.OfDouble C_DOUBLE = (ValueLayout.OfDouble) LINKER.canonicalLayouts().get("double");

I've improved the micro benchmarks with 3rd commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16179#discussion_r1358190616


More information about the hotspot-compiler-dev mailing list