RFR: 8332900: RISC-V: refactor nativeInst_riscv.cpp and macroAssembler_riscv.cpp [v3]

Hamlin Li mli at openjdk.org
Fri May 31 11:50:07 UTC 2024


On Fri, 31 May 2024 10:48:26 GMT, Fei Yang <fyang at openjdk.org> wrote:

> Like `NativeInstruction::is_li16u` which delegates work to `MacroAssembler::is_li16u_at`.

I don't find `NativeInstruction::is_li16u`, maybe you want to say something else for the delegation you mentioned?

Take `MacroAssembler::is_li16u_at` as example, I moved it to macroAssembler, because in macroAssembler it's used too. So one of the principals I'd like to stick to in this refactoring is to make these 2 classes's communication unidirectional, so maybe it's better to move `MacroAssembler::is_li16u_at` too.

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

PR Comment: https://git.openjdk.org/jdk/pull/19459#issuecomment-2141872299


More information about the hotspot-dev mailing list