<div dir="ltr"><div class="gmail_quote">On Mon, Apr 20, 2015 at 5:17 PM David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Apr 20, 2015 at 6:07 AM, Paweł Bylica <<a href="mailto:chfast@gmail.com" target="_blank">chfast@gmail.com</a>> wrote:<br>
> Hi LLVMdev,<br>
><br>
> I have a test that checks if CodeGen implements shift instruction for<br>
> illegal types correctly. The test itself checks the runtime result of the<br>
> operation using lli tool. This runtime check approach is quite different<br>
> from tests in test/CodeGen, as they use llc tool mainly.<br>
<br>
Why are you testing it that way, rather than the way other<br>
test/CodeGen tests work - verifying assembly?<br></blockquote><div><br></div><div>Before the bug was fixed I haven't known what assembly to expect. I can now write a test that checks the assembly but the test will be both target and implementation specific. Do you think it is better?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The regression suite generally (with some exceptions, like the JIT)<br>
doesn't execute newly-compiled code. We just test that we produce the<br>
right code, not that it executes correctly. (we do this for speed,<br>
isolation/stability, etc) We do have the "nightly test suite" (in the<br>
test-suite repository) that handles executing arbitrary code examples,<br>
etc - but that's mostly for large-ish programs with standalone<br>
functionality, not targeted tests.<br>
<br>
><br>
> Should my test be places somewhere else? Or should it be reimplemented?<br>
> <a href="http://reviews.llvm.org/D7752" target="_blank">http://reviews.llvm.org/D7752</a><br>
><br>
> - Paweł<br>
><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
><br>
</blockquote></div></div>