<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-size:12.8px">Why do you need this in support? It could be in Object, no?</span> </blockquote><div>I wasn't sure where exactly this should live until Jrmuizel suggested it should live in Support.</div><div>See here <a href="https://reviews.llvm.org/D29892">https://reviews.llvm.org/D29892</a><br>I believe it makes sense because the YAML Parser lives there also.<br><br>The def parser will be used by llvm-lib and llvm-dlltool and lld.<br>I wanted to update it in the future to give better parsing errors like the YAML Parser does.<br>Having them live together should help with readability.<br><br>If you have a strong Objection to this I can move it to Object also.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 1, 2017 at 2:36 PM, Rafael Avila de Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Martell Malone via Phabricator via llvm-commits<br>
<span class=""><<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> writes:<br>
<br>
> martell created this revision.<br>
> Herald added a subscriber: mgorny.<br>
><br>
> Repository:<br>
>   rL LLVM<br>
><br>
> <a href="https://reviews.llvm.org/D32689" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D32689</a><br>
><br>
> Files:<br>
>   include/llvm/Object/<wbr>COFFImportFile.h<br>
>   include/llvm/Support/<wbr>DEFParser.h<br>
>   lib/Object/CMakeLists.txt<br>
>   lib/Object/COFFImportFile.cpp<br>
>   lib/Support/CMakeLists.txt<br>
>   lib/Support/DEFParser.cpp<br>
<br>
</span>Why do you need this in support? It could be in Object, no?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>