RFR: JDK-8332423 : [PPC64] Remove C1_MacroAssembler::call_c_with_frame_resize [v9]

Martin Doerr mdoerr at openjdk.org
Tue Sep 3 22:13:22 UTC 2024


On Tue, 3 Sep 2024 16:13:57 GMT, Suchismith Roy <sroy at openjdk.org> wrote:

>> JBS Issue : [JDK-8332423](https://bugs.openjdk.org/browse/JDK-8332423)
>> C1_MacroAssembler::call_c_with_frame_resize is only used with frame_resize == 0. 
>> Also, call_c is adapted as per endianess of system. 
>> We can adapt the exisiting code to handle the endianness check at one place and not have to repeatedly check at multiple places to make calls to call_c.
>
> Suchismith Roy has updated the pull request incrementally with one additional commit since the last revision:
> 
>   indents

You effectively changed the type from `relocInfo::runtime_call_type` to `relocInfo::none` in c1_LIRAssembler_ppc.cpp. This is causing problems with ABIv1.
The VM seems to work when switching off C1 on AIX. So, the other files should be ok.

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

Changes requested by mdoerr (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19947#pullrequestreview-2278607902


More information about the hotspot-compiler-dev mailing list