RFR: 8339359: RISC-V: Use auipc explicitly in far_jump and far_call macro assembler routines

Fei Yang fyang at openjdk.org
Tue Sep 3 02:14:05 UTC 2024


On Mon, 2 Sep 2024 01:15:36 GMT, Fei Yang <fyang at openjdk.org> wrote:

> Previous discussion: https://github.com/openjdk/jdk/pull/18942#issuecomment-2109162337
> 
> For MacroAssembler::far_call and MacroAssembler::far_jump, I would suggest we use explicit
> auipc instead of MacroAssembler::la for them as the destination is ensured to be in code cache.
> This will help save unnecessary check in MacroAssembler::la and make the code more consistent.
> Also this will help distinguish these two macro assembler routines from MacroAssembler::rt_call
> 
> Testing:
> - [x] release & fastdebug builds
> - [x] Tiered 1-3 tests

Thanks for having a look. I just added one extra commit correcting the assertion message of far_jump.
My local tier1-3 tests are good. @robehn : Can you take another look and approve this?

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

PR Comment: https://git.openjdk.org/jdk/pull/20805#issuecomment-2325474428


More information about the hotspot-dev mailing list