<div dir="ltr">Probably sensible enough then to just make it a dumping test - I think that's generally 'better'/purist when adding dumper features, I just understand it's sometimes a bit more fuss. So I'll never complain about you doing it that way (checking a small sample source, a binary, etc - you can see the other pure dwarfdump tests).<br><br>& since you guys are planning to make this more of a production tool, I think it's a good idea to have more tests for dwarfdump itself - so that we don't accidentally make holes in the dwarfdump test coverage when we refactor tests.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 8, 2014 at 5:23 PM, Frederic Riss <span dir="ltr"><<a href="mailto:friss@apple.com" target="_blank">friss@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="HOEnZb"><div class="h5">>>! In D5689#4, @dblaikie wrote:<br>
> Looks reasonable (apart from testing) - depends how you want to do it. I assume there are some existing tests that test properties that could be updated to use this feature? (does this not regress any existing tests?)<br>
><br>
> If there aren't any such tests, I imagine we want some - and they could just test this incidentally.<br>
><br>
> (I'm usually on the fence about writing tool tests separately from unit tests... *shrug* sometimes it's nice, sometimes it seems like overkill)<br>
<br>
</div></div>There aren't any. I discovered the discrepancy between the enum values yesterday by trying to produce some tests and the results didn't actually make sense. I can add an llvm test, sure, but my ObjC foo being what it is, I fear that the code won't make any sense :-) Thus it won't test much except the actual dumping.<br>
<br>
<a href="http://reviews.llvm.org/D5689" target="_blank">http://reviews.llvm.org/D5689</a><br>
<br>
<br>
</blockquote></div><br></div>