[PATCH] D159082: [ELF][RISCV] Implement --emit-relocs with relaxation

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 5 14:42:48 PDT 2023


MaskRay added a comment.

I think I like R_RISCV_ALIGN better as it is simpler and conveys more information.
Changing to R_RISCV_NONE may not be necessarily their intention. It's just a default behavior in GNU ld when a relocation is nullified.

Ultimately, I think a user processing these

`--emit-relocs` copied relocations do not have the SHF_ALLOC flag, so they are easily recognizable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159082



More information about the llvm-commits mailing list