RFR: 8311630: [s390] Implementation of Foreign Function & Memory API (Preview) [v6]
sid8606
duke at openjdk.org
Mon Jul 31 08:04:30 UTC 2023
On Fri, 28 Jul 2023 12:36:13 GMT, Andrew Haley <aph at openjdk.org> wrote:
>> sid8606 has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Preserve and restore register Z_R6
>>
>> Though Z_R6 is argument register it is a saved register
>> so preserve and restore Z_R6 register
>
> src/hotspot/cpu/s390/upcallLinker_s390.cpp line 72:
>
>> 70: // Z_SP saved/restored by prologue/epilogue
>> 71: if (reg == Z_SP) continue;
>> 72: // though Z_R6 is argument register it is a saved register
>
> Suggestion:
> ` // although Z_R6 is used for parameter passing, it must be saved and restored by a called function.`
Thanks for the review. Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14801#discussion_r1278922263
More information about the core-libs-dev
mailing list