[PATCH] D132633: [LoongArch] Add more fixups and relocations

WÁNG Xuěruì via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 25 00:19:21 PDT 2022


xen0n accepted this revision.
xen0n added a comment.
This revision is now accepted and ready to land.

So it seems we're restricting the applicable instructions for each fixup kind, for example even if both `lu12i.w` and `lu32i.d` take a 20-bit signed immediate, the `xxx64_LO20` fixups still cannot be used with `lu12i.w` and the `xxx_HI20` ones cannot be used with `lu32i.d`.

Is this the same behavior as in binutils? I don't know if this is an artificial limitation to assembly expressiveness, but since I know the reloc types are deliberately designed with effectively only the linker implementation in mind, this might as well just be expected. I didn't find any other major problem though...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132633



More information about the llvm-commits mailing list