[PATCH] <<Proposed Driver Update>>

[Nick Kledzik]

On Mar 5, 2013, at 7:28 PM, Michael Spencer <bigcheesegs at gmail.com> wrote:
>  Sorry for the late reply, I've been at a conference. These changes in large are fine, but I still have a concern with not having a unified core. We loose the ability to use the same command line on a different system, as the driver may add search paths and settings based on the environment the driver is run in.
That is why I added:
>> Remaining work:
>> * Each TargetInfo class should have a dump() or print() method that can be used to print out what would
My thought was that each TargetInfo would have a dump() method that would print out a set of command line options to reproduce the TargetInfo.  The options should be not just echo the command line, but create a new canonical command line given just the TargetInfo, thus be independent of any env vars that were used to set up the TargetInfo.  This dump() method can also be used to help debug in-memory linking by printing out the linking options in a familiar (command line) format.


> It also makes lld -core pretty useless. I'm not dead set against loosing this functionality, but it needs to be worth it. The only thing we gain is having simpler option parsing code.
lld -core is still useful for unit tests that are testing the core linking algorithms.   If you are not testing core functionality, but say ELF functionality, then you need to use a driver (such as binutils) that handles ELF linking.

-Nick