[PATCH] D117082: [JITLink][RISCV] Support R_RISCV_SET relocations

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 11 19:29:57 PST 2022


jrtc27 added inline comments.


================
Comment at: llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp:276
+    }
+    case R_RISCV_32_PCREL: {
+      int64_t Value = E.getTarget().getAddress() + E.getAddend() - FixupAddress;
----------------
jrtc27 wrote:
> This is not an R_RISCV_SET* relocation, yet the commit message says that's that is added
*that's all that is added


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117082/new/

https://reviews.llvm.org/D117082



More information about the llvm-commits mailing list