<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Ok for these, that is probably a small fraction of the 162 test in here :)<div class=""><br class=""></div><div class="">In the meantime here are two patches to require “native” feature available for these tests (and others):</div><div class=""><br class=""></div><div class=""><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11730&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=FCRXEU4mDoolK8qBWeukfPT3i7YCxBfa2kMSoK3fczo&s=uOezJyAP1c2Xdwr4PvosHy--9bZO6sUcKNrh28-89Fw&e=" class="">http://reviews.llvm.org/D11730</a></div><div class=""><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11731&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=FCRXEU4mDoolK8qBWeukfPT3i7YCxBfa2kMSoK3fczo&s=mXVlZqlgp0XxcvP-whmzBxmhwqbTIyLKEZr2Bh9TXUg&e=" class="">http://reviews.llvm.org/D11731</a></div><div class=""><br class=""></div><div class="">(need reviewers)</div><div class=""><br class=""></div><div class="">Thanks,</div><div class=""><br class=""></div><div class="">— </div><div class="">Mehdi</div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Aug 3, 2015, at 12:48 PM, Alex L <<a href="mailto:arphaman@gmail.com" class="">arphaman@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Some of the tests are used to test the command line options in llc and the behavior of the LLVMTargetMachine class, like stop-after.ll and run-pass.ll. <span style="line-height:normal" class="">I think that they are expected to work on every target and should stay here.</span><div class=""><div class=""><br class=""></div><div class="">Cheers,</div><div class="">Alex</div></div></div><div class="gmail_extra"><br class=""><div class="gmail_quote">2015-08-03 12:22 GMT-07:00 Mehdi Amini <span dir="ltr" class=""><<a href="mailto:mehdi.amini@apple.com" target="_blank" class="">mehdi.amini@apple.com</a>></span>:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br class="">
<br class="">
I wonder about this directory and the 162 tests that are in it.<br class="">
<br class="">
Most tests have only this run line: ; RUN: llc < %s<br class="">
<br class="">
And when I look at the log of some tests like: test/CodeGen/Generic/2002-04-14-UnexpectedUnsignedType.ll<br class="">
The comment inside is:<br class="">
<br class="">
; This caused the backend to assert out with:<br class="">
; SparcInstrInfo.cpp:103: failed assertion `0 && "Unexpected unsigned type”’<br class="">
<br class="">
I doubt we are effectively testing the Sparc backend unless you run the validation on a Sparc.<br class="">
<br class="">
Since “CodeGen” can’t really be generic (you’ll need a target), I’m not sure what is intended to be in it. I’d like to move most of the tests in their associated backend directory, but I’d like to hear what is legit to be here?<br class="">
<br class="">
Thanks,<br class="">
<br class="">
Mehdi<br class="">
<br class="">
<br class="">
<br class="">
<br class="">
<br class="">
<br class="">
_______________________________________________<br class="">
LLVM Developers mailing list<br class="">
<a href="mailto:LLVMdev@cs.uiuc.edu" class="">LLVMdev@cs.uiuc.edu</a> <a href="http://llvm.cs.uiuc.edu/" rel="noreferrer" target="_blank" class="">http://llvm.cs.uiuc.edu</a><br class="">
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" rel="noreferrer" target="_blank" class="">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br class="">
</blockquote></div><br class=""></div>
</div></blockquote></div><br class=""></div></body></html>