[PATCH] [ELF] Support for parsing OUTPUT command in LinkerScript
Davide Italiano
davide at freebsd.org
Sun Feb 1 20:44:33 PST 2015
In http://reviews.llvm.org/D7326#116598, @ruiu wrote:
> It basically looks good, but we already have a large piece of code as a part of the linker script parser which don't have any consumer. This patch adds a parser for OUTPUT() but you didn't make any change to use or interpret that directive. So, the new code doesn't actually provide value to the user by itself. It's also a little bit concerning -- there might be a design flaw that's only noticeable when we try to use the parser.
>
> Could you update the GNU driver to consume OUTPUT directive?
My plan was to split parsing and action, i.e. to implement the latter in a subsequent commit (was planning to do this tonight or tomorrow). If you have any concern I can incorporate the driver update in this change but I don't see a strong reason why they can't be committed as standalone changes. Let me know what you think about it.
http://reviews.llvm.org/D7326
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list