[PATCH] [lld][inputGraph] Implement flavor specific resolver

Rui Ueyama ruiu at google.com
Mon Oct 7 21:59:03 PDT 2013


On Mon, Oct 7, 2013 at 9:32 PM, Shankar Easwaran <shankare at codeaurora.org>wrote:

> On 10/7/2013 11:18 PM, Rui Ueyama wrote:
>
>> My point is the way to abstract linker input as InputGraph and
>> InputElement itself can be simplified.
>>
>>> Isnt the current model simplistic ?
>>>
>> Honestly, no, I feel that it's over engineerded a bit and designed to be
>> too generic. It may have been unavoidable with the previous design but we
>> may be able to simplify it with nextFile() infrastructure.
>>
> nextFile is intended as a way to model only the resolver. The command line
> has to be still represented in terms of a Input Graph(ELF linker has
> different variations and the input graph design still doesnot achieve
> completeness!).
>

The thing I'd like to see addressed is that we have too many classes to
represent input files and their relationships. They are abstract enough to
represent any possible linker inputs, especially because of ControlNode
which can become any directive, I think we don't really need and want that
power. It feels that the right amount of abstraction is less than that.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131007/8ae62119/attachment.html>


More information about the llvm-commits mailing list