[llvm-commits] [lld] r155183 - in /lld/trunk: include/lld/Core/ lib/Core/ test/ test/darwin/ tools/lld-core/

[Michael Spencer]

On Fri, Apr 20, 2012 at 6:05 PM, Nick Kledzik <kledzik at apple.com> wrote:
> On Apr 20, 2012, at 11:43 AM, Michael Spencer wrote:
>>> +---
>>> +kind: archive
>>> +members:
>>> +    - name:    bar.o
>>> +    - name:    baz.o
>>> +...
>>> +
>>> +# CHECK:       name:       foo
>>> +# CHECK:       name:       bar
>>> +# CHECK-NOT:   definition: undefined
>>> +# CHECK:       name:       bar2
>>> +# CHECK-NOT:   name:       baz
>>> +# CHECK:       ...
>>
>> I find this a weird way to represent archives in YAML. Each document
>> should represent a separate main file
>> handed to the linker, so all archive members should be in the same
>> document.  I would rather see:
>>
>> ---
>> archive:
>>  - name: bar.o
>>    atoms:
>>      - blah....
>>  - name: baz.o
>>      - blah....
>> ...
> It was easier to implement this way with the existing simple parser.  When do you think your yaml parser will be available?
>
> -Nick

I'm in the process of switching the code over to it.

- Michael Spencer