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

[Martin Doerr]

On Tue, 3 Sep 2024 15:59:54 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 two additional commits since the last revision:
> 
>  - cast
>  - casting

src/hotspot/cpu/ppc/macroAssembler_ppc.hpp line 371:

> 369:     return call_c((const FunctionDescriptor*)function_entry, rt);
> 370:   }
> 371:     // For tail calls: only branch, don't link, so callee returns to caller of this function.

This looks messed up! I requested to move it down, not do duplicate any function declaration. Also, the comment indentation is bad.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19947#discussion_r1742320366