[PATCH] D85430: [llvm-readobj] - Refine logic of the symbol table locating in printRelocationsHelper().

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 10 03:22:31 PDT 2020


grimar added a comment.

In D85430#2205933 <https://reviews.llvm.org/D85430#2205933>, @jhenderson wrote:

> I think the original SHT_RELR proposal was mistaken to say there's an associated symbol table, since as you've already highlighted, it represents relative relocations. In fact, later versions of the proposal dropped this (see e.g. https://groups.google.com/g/generic-abi/c/bX460iggiKg/m/Jnz1lgLJAgAJ):

Ok, I've also supposed there is some mistake, but was not 100% sure. Thanks for the link, I've missed that `sh_link` description was dropped later.
I'll update the implementation after landing the parent diff (D85303 <https://reviews.llvm.org/D85303>).


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

https://reviews.llvm.org/D85430



More information about the llvm-commits mailing list