[PATCH] D32689: DEF: migrate def parser from LLD to LLVM

[Rafael Avila de Espindola via llvm-commits]

Martell Malone <martellmalone at gmail.com> writes:

>>
>> Why do you need this in support? It could be in Object, no?
>
> I wasn't sure where exactly this should live until Jrmuizel suggested it
> should live in Support.
> See here https://reviews.llvm.org/D29892
> I believe it makes sense because the YAML Parser lives there also.
>
> The def parser will be used by llvm-lib and llvm-dlltool and lld.
> I wanted to update it in the future to give better parsing errors like the
> YAML Parser does.
> Having them live together should help with readability.

The yaml parser is used by tools that don't link with Object. All the
tools you listed use Object, no?

Cheers,
Rafael