[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