[PATCH] D41277: Implement --just-symbols
Chih-Mao Chen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 18 23:13:25 PST 2017
PkmX updated this revision to Diff 127465.
PkmX added a comment.
Added a check to only allow `ET_EXEC` and the corresponding test.
On Tue, Dec 19, 2017 at 9:54 AM, Rafael Avila de Espindola <rafael.espindola at gmail.com> wrote:
> This patch allows --just-symbols to be a ET_DYN, ET_REL or ET_EXEC. Do
> you need all 3? ET_REL in particular seems problematic since st_value
> has a different meaning in relocatable files.
> If you don't need ET_DYN or ET_REL, please reject them. If you need
> them, please add a test.
`ld.bfd` accepts `ET_EXEC` and `ET_REL` but not `ET_DYN`. However I don't think this option makes much sense for `ET_REL` inputs so I'm restricting it to `ET_EXEC` for now.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6716 bytes
Desc: not available
More information about the llvm-commits