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

Fei Yang fyang at openjdk.org
Tue Aug 30 01:22:05 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

Thanks for taking care of riscv-specific part at the same time. I guess you might need some rework since https://github.com/openjdk/jdk/pull/10056 is integrated.

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

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


More information about the hotspot-dev mailing list