RFR: 8294087: RISC-V: RVC: Fix a potential alignment issue and add more alignment assertions for the patchable calls/nops [v4]

[Aleksey Shipilev]

On Wed, 21 Sep 2022 12:24:48 GMT, Xiaolin Zheng <xlinzheng at openjdk.org> wrote:

>> src/hotspot/cpu/riscv/nativeInst_riscv.cpp line 274:
>> 
>>> 272: 
>>> 273:   // Must be 4 byte aligned
>>> 274:   MacroAssembler::assert_alignment(verified_entry);
>> 
>> So, if the intent to pass explicit `NativeInsn::insn_size` everywhere we say "Must be 4 bytes aligned", then we should probably pass it here too. Or, drop the explicit `NativeInsn::insn_size` everywhere else?
>
> Well, if I understand correctly, as we want to make it clean, we won't explicitly pass `NativeInstruction::instruction_size` as an argument of `__ assert_alignment()`. And it seems currently all the call sites of `__ assert_alignment()` have already dropped the explicit argument, including this position. So... haven't we already succeeded in dropping them?
> 
> The `__ align()` API needs an explicit argument as alignment, so when calling that one, the `NativeInstruction::instruction_size` is passed explicitly.

Sorry, I misread the patch, and confused `align` and `assert_alignment`.

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

PR: https://git.openjdk.org/jdk/pull/10370