[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
Wed May 9 21:52:10 PDT 2018


asb added a comment.

Hi Shiva. Could you please split out the definition of shouldForceRelocation, enableLinkerRelax and FeatureRelax to a separate patch. That way we can commit that followed by all the patches that rely on querying whether linker relaxation is enabled, and then finally enable it via the RISCVMCCodeEmitter changes in this patch. Now there are a few patches we know we want to commit before linker relaxation is actually safe to use, I think that's probably the best way to structure things.


Repository:
  rL LLVM

https://reviews.llvm.org/D44886





More information about the llvm-commits mailing list