RFR: 8364120: RISC-V: unify the usage of MacroAssembler::instruction_size

Fei Yang fyang at openjdk.org
Mon Jul 28 03:59:00 UTC 2025


On Fri, 25 Jul 2025 14:00:54 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review this simple cleanup?
> 
> in MacroAssembler, some magic number 4 is used for MacroAssembler::instruction_size. We should not use magic number if possible.
> This pr also replaces all the usage of  `instruction_size` with `MacroAssembler::instruction_size`.
> 
> Thanks

Looks fine. Thanks.

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

Marked as reviewed by fyang (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26481#pullrequestreview-3060245312


More information about the hotspot-dev mailing list