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.