<div dir="ltr">So, I'd like to get some input on how to test this patch.<div><br></div><div>The new file format is used basically only when there are more than 2^16 relocations in a file. It's easy to create a file containing lots of relocations using .rept macro in assembly, but such a file in an object file format could be large (~2MB). I don't think we want to check in such large binary file to the repository.</div>

<div><br></div><div>One workaround would be to check in only the assembly file, and run <font face="courier new, monospace">as</font> when test is run. Does that make sense? Is there a best practice for this?</div></div>
<div class="gmail_extra">
<br><br><div class="gmail_quote">On Mon, Nov 18, 2013 at 9:23 PM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

  Updated as per Nick's comments.<br>
<br>
  - Resurrect V1 format<br>
  - Use V2 format only when the resulting file cannot be represented in V1<br>
<br>
Hi kledzik, Bigcheese, shankarke,<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2217" target="_blank">http://llvm-reviews.chandlerc.com/D2217</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
  <a href="http://llvm-reviews.chandlerc.com/D2217?vs=5641&id=5647#toc" target="_blank">http://llvm-reviews.chandlerc.com/D2217?vs=5641&id=5647#toc</a><br>
<br>
Files:<br>
  lib/ReaderWriter/Native/NativeFileFormat.h<br>
  lib/ReaderWriter/Native/ReaderNative.cpp<br>
  lib/ReaderWriter/Native/WriterNative.cpp<br>
</blockquote></div><br></div>