<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 29, 2014 at 11:56 AM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
> On 2014-Jul-29, at 09:43, Sean Silva <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>
><br>
> Some random comments:<br>
><br>
> 1. Looks like the comment at the top of the file is copypasta from opt.<br>
<br>
</div>r214187, thanks for noticing.<br>
<div class=""><br>
> 2. The comment at the top of this file should describe the purpose of this tool. From skimming the code it looks like some sort of fuzzer.<br>
<br>
</div>Did something quick in r214187 -- sufficient or do you think there should be<br>
more?<br></blockquote><div><br></div><div>Maybe some information about invocation and basic usage?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class=""><br>
> 3. The name is incredibly ambiguous; could it at least have a verb in it? It seems like it is a fuzzer that verifies that use-lists correctly round-trip through bitcode. Is there a better name? Maybe something with "verify" in it? (a long-ish name for a specialized tool like this wouldn't seem that out of place).<br>

<br>
</div>Here are two options:<br>
<br>
  - verify-uselistorder<br>
  - llvm-verify-uselistorder<br>
<br>
Either of those sound better to you?<br>
</blockquote></div><br></div><div class="gmail_extra">Both sound fine. Take your pick.</div><div class="gmail_extra"><br></div><div class="gmail_extra">-- Sean Silva</div></div>