<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 20, 2015 at 7:59 AM, Shankar Easwaram <span dir="ltr"><<a href="mailto:shankarke@gmail.com" target="_blank">shankarke@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I plan to add a test case but couldn't add because of yaml2obj support. I am adding that support to yaml2obj and obj2yaml and push this patch later. I don't like the idea of adding binary files to the test repo as we need to keep cleaning up later.<br></blockquote><div><br></div><div>Please make sure to pre-commit review this. I expect a bikeshed over the syntax.</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
> On Apr 20, 2015, at 03:34, Simon Atanasyan <<a href="mailto:simon@atanasyan.com">simon@atanasyan.com</a>> wrote:<br>
><br>
>> On Mon, Apr 20, 2015 at 1:44 AM, Shankar Easwaran <<a href="mailto:shankarke@gmail.com">shankarke@gmail.com</a>> wrote:<br>
>> Attached patch fixes the issue.<br>
><br>
> Thanks for the quick fix. The patch LGTM if you add a test case.<br>
> Unfortunately yaml2obj does not support duplicated section names but<br>
> we can use a binary input file.<br>
><br>
> --<br>
> Simon Atanasyan<br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</div></div></blockquote></div><br></div></div>