[PATCH] [ELF] Support for parsing OUTPUT command in LinkerScript

Rui Ueyama ruiu at google.com
Sun Feb 1 20:35:32 PST 2015


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?


http://reviews.llvm.org/D7326

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list