[PATCH] D81359: [ELF] Add --[no-]relax to disable x86-64 GOTPCRELX relaxation

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 8 08:48:19 PDT 2020


grimar added a comment.

In D81359#2079839 <https://reviews.llvm.org/D81359#2079839>, @MaskRay wrote:

> In D81359#2079188 <https://reviews.llvm.org/D81359#2079188>, @grimar wrote:
>
> > Can `--no-relax` just be an ignored option for now? Seems it is only valueable for RISC-V.
>
>
> The implementation for x86-64 seems simple enough that we can implement it as well.


But:

1. No implementing it, but ignoring is a single line in options.td I think.
2. `--no-relax` name is probably itself confusing because of TLS relaxations that are not affected.
3. It still introduce some code but there is no evidence it is usefull.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81359





More information about the llvm-commits mailing list