RFR: 8294187: RISC-V: Unify all relocations for the backend into AbstractAssembler::relocate() [v2]

[Xiaolin Zheng]

On Fri, 23 Sep 2022 02:55:35 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Xiaolin Zheng has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Migrate them to the next patch for AbstractAssembler has their counterparts
>>  - Remove useless add_debug_info_for_branch(adr, CodeEmitInfo*)
>
> 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);
>   }

Thanks for pointing this out. All done.

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

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