[PATCH] D44886: [RISCV] Support linker relax function call from auipc and jalr to jal
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 12 06:22:17 PDT 2018
asb added a comment.
Given that Simon's https://reviews.llvm.org/D45181 will be dependent on this (as it should really query whether linker relaxation is enabled), it probably makes sense to get this merged sooner rather than later.
I think the options are:
1. Merge as is
2. Modify this patch so it adds RISCV_RELAX to pcrel_hi and pcrel_lo relocations when linker relaxation is enabled, then have a follow-up patch that emits CALL.
The psabi doc doesn't discuss attaching RISCV_RELAX to relocations other than RISCV_CALL or RISCV_CALL_PLT, but presumably binutils ld will use it for other pairs?
Repository:
rL LLVM
https://reviews.llvm.org/D44886
More information about the llvm-commits
mailing list