On Tue, 23 May 2023 12:26:52 GMT, Martin Doerr wrote:
> I believe omitting the PSA is wrong for varargs, but we don't have this
> information in the backend.
It is clearly wrong.
> So, I think we should simply not optimize it.
Already agreed for this version.
> Reserving 64 Byte stack space
On Tue, 23 May 2023 07:46:08 GMT, Richard Reingruber wrote:
>> src/hotspot/cpu/ppc/downcallLinker_ppc.cpp line 163:
>>
>>> 161: // The Parameter Save Area needs to be at least 8 slots for ABIv1.
>>> 162: // ABIv2 allows omitting it when all parameters can get passed in
>>> registers. We cur
On Mon, 22 May 2023 22:44:41 GMT, Martin Doerr wrote:
> Thanks for publishing our discussion, here. The unnecessary PSA affects other
> areas of hotspot much more than Panama. Yes, we should file an RFE. I think
> one for hotspot is sufficient as the downcall stub is part of it. I don't
> thin
On Tue, 23 May 2023 07:37:37 GMT, Richard Reingruber wrote:
>> Martin Doerr has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Parameter Save Area is the correct name.
>
> src/hotspot/cpu/ppc/downcallLinker_ppc.cpp line 163:
>
>> 161: //
On Mon, 22 May 2023 22:29:18 GMT, Martin Doerr wrote:
>> Implementation of "Foreign Function & Memory API" for linux on Power (Little
>> Endian) according to "Power Architecture 64-Bit ELF V2 ABI Specification".
>>
>> This PR does not include code for VaList support because it's supposed to
>>
On Mon, 22 May 2023 22:29:18 GMT, Martin Doerr wrote:
>> Implementation of "Foreign Function & Memory API" for linux on Power (Little
>> Endian) according to "Power Architecture 64-Bit ELF V2 ABI Specification".
>>
>> This PR does not include code for VaList support because it's supposed to
>>
> Implementation of "Foreign Function & Memory API" for linux on Power (Little
> Endian) according to "Power Architecture 64-Bit ELF V2 ABI Specification".
>
> This PR does not include code for VaList support because it's supposed to get
> removed by [JDK-8299736](https://bugs.openjdk.org/browse
