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

[Nick Kledzik]

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