<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 27, 2014 at 10:21 AM, James Courtier-Dutton <span dir="ltr"><<a href="mailto:james.dutton@gmail.com" target="_blank">james.dutton@gmail.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="im">On 27 January 2014 17:24, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>

> Looks like this was committed (without a test case) by Craig in 200100.<br>
><br>
> Craig - any reason it's missing a test case?<br>
><br>
>>><br>
</div><div class="im">>>> Is there any chance of getting this bug fixed?<br>
>>> <a href="http://llvm.org/bugs/show_bug.cgi?id=17310" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=17310</a><br>
>>><br>
>>> It is a very simple 2 line patch (see below).<br>
>>> There is also a test case in the bug report.<br>
>>><br>
<br>
</div>There is a test case in the bug report.<br>
It tests the disassembler callback API. Something that is not covered<br>
in any other tests.<br>
So, I cannot simply add an extra instruction to the current filecheck tests.<br>
<br>
I would need some guidance as to how to modify the current test I have<br>
written to fit with the LLVM automatic testing.<br>
The test I have written submits hex bytes to the disassembler, and<br>
then checks that the correct callbacks were called with the correct<br>
parameters during the disassemble/decode of the instruction.<br></blockquote><div><br></div><div>We do have unit tests as well for arbitrary API-level testing (see llvm/unittests) - though I don't know how amenable this particular API is to unit testing. If it's moderately convenient it'd be great to add such a test. If it requires a lot of engineering to make the API unittestable then that burden isn't necessarily on you to do.<br>
<br>- David </div></div></div></div>