<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 19, 2012 at 4:40 PM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi All,<br>
<br>
I'm attaching a patch for llvm-objdump which adds the<br>
-no-show-raw-insn option from objdump (do not print the raw<br>
instruction encoding when disassembling). Please take a look.<br>
<br>
The patch itself is pretty simple but it raises an interesting concern<br>
about the lack of tests targeted at llvm-objdump (and possibly other<br>
tools we have). I assume there won't be much disagreement on whether<br>
we should have tests for the tools at all, but where should we put<br>
them? AFAICS, test/tools does not exist. Should it be added?<br></blockquote><div><br></div><div>Can, I don't really have an objection. For my uses I've been largely adding them in the area for things they're designed to test, but in this case you're testing the tool itself so that sounds good to me. Other option is test/Object and start adding tests with input there.</div>
<div><br></div><div>I'm neutral on which one I'd prefer so it's up to you.</div><div><br></div><div>-eric</div></div></div>