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

Mon Aug 29 15:24:48 UTC 2022

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

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

Commit messages:
 - 8293035: Cleanup MacroAssembler::movoop code patching logic aarch64 riscv

Changes: https://git.openjdk.org/jdk/pull/10065/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=10065&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8293035
  Stats: 35 lines in 12 files changed: 6 ins; 11 del; 18 mod
  Patch: https://git.openjdk.org/jdk/pull/10065.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10065/head:pull/10065

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