RFR: 8294187: RISC-V: Unify all relocations for the backend into MacroAssembler::relocate()

[Fei Yang]

On Thu, 22 Sep 2022 06:56:05 GMT, Xiaolin Zheng <xlinzheng at openjdk.org> wrote:

> Unify all relocations into MacroAssembler::relocate() as a simple cleanup, so that there are platform-related entrances for them. We will have needs to insert some code into the new MacroAssembler::relocate() afterward. Also, the (__ pc()) everywhere seems a little verbose here.
> 
> Tested hotspot tier1 and tier2 on QEMU.

Changes requested by fyang (Reviewer).

src/hotspot/cpu/riscv/c1_LIRAssembler_riscv.cpp line 2147:

> 2145: }
> 2146: 
> 2147: void LIR_Assembler::add_debug_info_for_branch(address adr, CodeEmitInfo* info) {

I see this function is never used: void LIR_Assembler::add_debug_info_for_branch(address adr, CodeEmitInfo* info)
So I would suggest remove this function. Then following two functions introduced by this PR would become unnecessary:
  void relocate(address at, RelocationHolder const& rspec, int format = 0) {
    code_section()->relocate(at, rspec, format);
  }
  void relocate(address at, relocInfo::relocType rtype, int format = 0, jint method_index = 0) {
    code_section()->relocate(at, rtype, format, method_index);
  }

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

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