[PATCH] D80496: [ELF] Add -z rel and -z rela

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 26 13:05:53 PDT 2020


MaskRay added a comment.

In D80496#2054133 <https://reviews.llvm.org/D80496#2054133>, @grimar wrote:

> LGTM with a last comment addressed.
>  Probably hold this a bit to give other people chance to look too.


This may be a display problem. In the latest revision https://reviews.llvm.org/D80496?id=266043 , the files are i386-zrel-zrela.s and x86-64-zrel-zrela.s , but Phabricator seems to display the old names (probably to associate with the old comments).

I'll definitely wait a bit longer to get more input. FWIW I also asked binutils and glibc https://sourceware.org/pipermail/binutils/2020-May/111244.html (it may be difficult for them due to how they organize code...)
asked FreeBSD, mentioned it in a kernel thread, etc...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80496





More information about the llvm-commits mailing list