[PATCH] D47493: [llvm-objdump] Add -R option

Paul Semel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 6 15:48:43 PDT 2018


paulsemel added a comment.

In https://reviews.llvm.org/D47493#1124195, @echristo wrote:

> Some of the logic in the function feels awkward, but that's a cleanup for another time probably.
>
> -eric


Yes, I didn't really know how to do it better. I truly think that we want to output the sections and not directly the relocations.
This is why there is this weird logic (there is no pointer to reloc section in the Dynamic Section).
If you find any better way to do it, this might be awesome !
Will land tomorrow :)


Repository:
  rL LLVM

https://reviews.llvm.org/D47493





More information about the llvm-commits mailing list