[PATCH] D142278: [LoongArch] Allow %pc_lo12 relocs in JIRL's immediate operand position

Sat Jan 21 05:40:52 PST 2023

xen0n added a comment.

In D142278#4071050 <https://reviews.llvm.org/D142278#4071050>, @xry111 wrote:

> By the way, using PCALA_LO12 like this means `lld` will need to peek at the instruction and see if it's a `jirl` to handle the PCALA_LO12 relocation properly.  I guess we'll need to investigate this issue a little.  I hope it won't cause too much trouble.

I've checked and there is one (and only one) similar case: X86 `R_386_GOT32` and `R_386_GOT32X` need to read the input section too. At least there //is// precedent so we're technically not alone... and we're only a PR away from having this kludge documented.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142278