<div dir="ltr">Hi Tobi,<div><br></div><div>Sorry for missing you off the to list!</div><div><br></div><div><span style="font-family:arial,sans-serif;font-size:13px">> Also, James, did you happen to run some performance test with the new order?</span><br>
</div><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">Not yet. I've got one queued to run over the weekend.</span></div><div>
<span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">Cheers,</span></div><div><span style="font-family:arial,sans-serif;font-size:13px"><br>
</span></div><div><font face="arial, sans-serif">James</font></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 25 July 2014 16:48, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</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">On 25/07/2014 17:41, James Molloy wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Nadav, Arnold,<br>
<br>
<br>
<br>
I've come across an interesting optimization problem in one of the SPEC<br>
benchmarks. There is a loop that can be optimized by both the SLP vectorizer<br>
and the loop vectorizer (when I patch the loop vectorizer to deal with fsub<br>
reductions).<br>
<br>
<br>
<br>
The SLP vectorizer actually makes the performance worse - I think this is<br>
due to a lack of loop unrolling afterwards. The Loop vectorizer can improve<br>
the performance.<br>
<br>
<br>
<br>
However, the loop vectorizer runs after the SLP vectorizer, so it never gets<br>
a chance. I'd have thought the ideal order would be Loop Vectorizer -> SLP<br>
vectorizer -> BB vectorizer, given that the loop vectorizer if it can run<br>
will probably give greater speedup than SLP.<br>
<br>
<br>
<br>
The current sequence is SLP vectorizer -> BB vectorizer -> Loop vectorizer.<br>
</blockquote>
<br></div></div>
Even though I was not directly addressed. I still reply.<br>
<br>
The proposed new order is what I think makes most sense. I wonder what was the reason to go for the current order. Nadav, Arnold did you choose this order?<br>
<br>
Also, James, did you happen to run some performance test with the new order?<br>
<br>
Cheers,<br>
Tobias<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>