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

Roland McGrath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 20 11:23:22 PDT 2020


mcgrathr added a comment.

I think if you provide symbols for the section bounds, they should be `__start_note_gnu_build_id` and `__stop_note_gnu_build_id`.  But that doesn't seem enough of an improvement over parsing phdrs from `__ehdr_start` to merit the feature IMHO.  Consumers still have to parse (or worse yet, assume) the note headers.  What would make things trivial for consumers is a pair of symbols giving exactly the bounds of the *payload*, i.e. just the build ID itself (and presenting its variable size in a simple manner).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76482





More information about the llvm-commits mailing list