[PATCH] D76482: [lld][ELF] Provide optional hidden symbols for build ID

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 30 14:05:12 PDT 2020


MaskRay added a comment.

In D76482#2185833 <https://reviews.llvm.org/D76482#2185833>, @mcgrathr wrote:

> The proposed solution with an input linker script is a clever kludge, but it's a kludge.  There's no way to make that actually general, since there is no particular named section that you can correctly assume is present in all kinds of binaries you might be linking.  The kludge could be locally tailored for different cases, but that is even worse.  Since the section and layout we're talking about here is entirely synthesized by the linker, it makes a lot more sense to have the linker support convenience features for using that format directly than to require users to build detailed kludges baking in the details of the linker's built-in synthesizing behavior.

This may worth a discussion on binutils. There can be several alternative syntax which may meet your needs.


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

https://reviews.llvm.org/D76482



More information about the llvm-commits mailing list