RFR: 8293035: Cleanup MacroAssembler::movoop code patching logic aarch64 riscv

Erik Österlund eosterlund at openjdk.org
Mon Aug 29 15:38:07 UTC 2022


On Mon, 29 Aug 2022 15:16:23 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

> Cleanup MacroAssembler::movoop logic with respect to code patching on aarch64 and riscv.
> 
> Remove the `bool immediate` argument as it is only used with the value `true` in `void MacroAssembler::movoop(Register dst, jobject obj, bool immediate)`
> 
> Make comments and logic more clear and easier to read.
> 
> Testing: osx, linux aarch64 tier 1-3

Looks good. Thanks for fixing!

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

Marked as reviewed by eosterlund (Reviewer).

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


More information about the hotspot-dev mailing list