RFR: 8303040: linux PPC64le: Implementation of Foreign Function & Memory API (Preview) [v5]
Martin Doerr
mdoerr at openjdk.org
Wed Mar 1 06:13:03 UTC 2023
On Wed, 22 Feb 2023 20:25:07 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> Martin Doerr has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Minor cleanup.
>
> src/java.base/share/classes/jdk/internal/foreign/abi/ppc64/CallArranger.java line 81:
>
>> 79: new VMStorage[] { f1, f2, f3, f4, f5, f6, f7, f8 }, // FP output
>> 80: new VMStorage[] { r0, r2, r3, r4, r5, r6, r7, r8, r9, r10, r11, r12 }, // volatile GP
>> 81: new VMStorage[] { f0, f1, f2, f3, f4, f5, f6, f7, f8, f9, f10, f11, f12, f13 }, // volatile FP
>
> Note that argument registers are assumed volatile, so they don't have to be duplicated here.
Removed.
> src/java.base/share/classes/jdk/internal/foreign/abi/ppc64/linux/LinuxPPC64CallArranger.java line 33:
>
>> 31: * PPC64 CallArranger specialized for Linux ABI.
>> 32: */
>> 33: public class LinuxPPC64CallArranger extends CallArranger {
>
> I don't really see the point in having a separate subclass with `CallArranger` being abstract, unless you are planning to add other implementations later?
>
> (edit: see also later comment in CallArranger https://github.com/openjdk/jdk/pull/12708#discussion_r1120753657)
AIX support will need to get implemented, yet. I guess @backwaterred will work on it.
-------------
PR: https://git.openjdk.org/jdk/pull/12708
More information about the core-libs-dev
mailing list