<div dir="ltr">New to arc, I don't know how to help with downloading the patch.<div>Github alternative (the top-level cmake changes are missing, there are just 2 modified lines, please pull that from Phabricator):</div><div><a href="https://github.com/alinas/Bitcode">https://github.com/alinas/Bitcode</a><br></div><div><br></div><div><br></div><div>Thanks,</div><div>Alina</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 29, 2016 at 11:40 AM, Mehdi Amini <span dir="ltr"><<a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@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 style="word-wrap:break-word"><br><div><span class=""><blockquote type="cite"><div>On Feb 29, 2016, at 11:16 AM, Alina Sbirlea via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:</div><br><div><div dir="ltr">All,<div><br></div><div>To get the discussion going in a focused manner, here is an initial patch with a running test. The test is from the Halide suite and is checking the correctness of several simd operations.</div><div>(Notes: the patch is large due to the number of operations being tested; </div><div>I expect a lot of changes before actually landing it, this is simply to continue the discussion using a concrete example.)<br></div><div><a href="http://reviews.llvm.org/D17726" target="_blank">http://reviews.llvm.org/D17726</a><br></div></div></div></blockquote><div><br></div></span><div>I can't figure how to download the patch *with the bitcode files* from Phabricator. Can you push this on github (or somewhere else)? (or if I missed how to proceed...).</div><div><br></div><div>Thanks,</div><div><br></div><div>Mehdi</div><div><br></div><div><br></div><br><blockquote type="cite"><div><span class=""><div dir="ltr"><div><br></div><div>A few questions/todos to start the discussion:</div><div>1. What is a good location for these tests? They are in a separate Bitcode directory atm, but using the llvm_multisource. This may change to more closely model the approach for external tests (see next item). </div><div>2. There is a single .cpp file testing all operations provided by individual bitcode files. I expect this to change. Instead of using llvm_multisource to have the same test run with specific arguments, each run testing a single operation.</div><div>3. The building approach I took is to first link all bitcode files into a single one, then obtain the assembly for it, which cmake knows to take as an input source. </div><div>A separate discussion is on reading metadata (mcpu and mattr) in llc. I added a script to work around that for now.</div><div><br></div><div>Looking forward to your feedback!</div><div><br></div><div>Thanks,<br></div><div>Alina</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 19, 2016 at 6:50 AM, Kristof Beyls <span dir="ltr"><<a href="mailto:kristof.beyls@arm.com" target="_blank">kristof.beyls@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000"><span>
<br>
<br>
<div>On 18/02/2016 19:12, Alina Sbirlea via
llvm-dev wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr"><br>
<div class="gmail_extra">
<div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I have more questions for Alina. What kind of tests do you
have:<br>
<br>
- "the compiler takes the bitcode and generates code
without crashing"<br>
- "the compiled test runs without crashing"<br>
- "the compiled test will produce an output that be
checked against a reference"<br>
- "the compiled test is meaningful as a benchmarks"<br>
</blockquote>
<div><br>
</div>
<div>We have all 4 kinds of tests in Halide. The bitcode
files for the first category is already available and I'm
working on building the ones for the next 3. We'd like to
include all incrementally.</div>
<div> </div>
<br>
</div>
</div>
</div>
</blockquote></span>
It seems to me that the first category ("the compiler takes the
bitcode and generates code without crashing") are tests that should
be added to the "make check-all" tests in the LLVM subproject,
rather than the test-suite subproject?<br>
Or if these tests currently don't crash the compiler anymore, the
bugs must have been fixed, and there should already be equivalent
tests?<br>
</div>
</blockquote></div><br></div></span><span class="">
_______________________________________________<br>LLVM Developers mailing list<br><a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br></span></div></blockquote></div><br></div></blockquote></div><br></div>