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

Hamlin Li mli at openjdk.org
Mon Jun 3 11:42:28 UTC 2024


On Mon, 3 Jun 2024 11:03:35 GMT, Robbin Ehn <rehn at openjdk.org> wrote:

> ```
> class Assembler : public AbstractAssembler {
> public:
> 
>   enum {
>     instruction_size = 4,
> 
> class MacroAssembler: public Assembler {
> 
>  public:
>   enum {
>     instruction_size = 4,
> ```
> 
> Not sure why you have the same enum defs in both asm and masm?

Yeh, it's redundant, I'll remove it. Thanks.

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

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


More information about the hotspot-dev mailing list