<div dir="ltr">Hi,<div><br></div><div>Ah yes - you asked about benchmarking numbers. LNT and Spec showed no real changes - there were a couple of modest improvements in LNT. For this patch I would expect the biggest benefit in small, hot loops using i8 or i16 types. This patch doesn't *enable* vectorization for these types, it just improves the cost model and makes sure we use the right vector types in all cases. In many trivial cases we were already vectorizing assuming i32 promotion and InstCombine was removing the promotion afterwards.</div><div><br></div><div>Image filters are the best example of code that will improve, and we don't have any in LNT or Spec. I have examples of image filters in the test-suite. I do have examples in third-party test suites that show upwards of 100% improvement with this patch.</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 12 Oct 2015 at 09:40 James Molloy via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jmolloy updated this revision to Diff 37083.<br>
jmolloy added a comment.<br>
<br>
Hi Silviu,<br>
<br>
Thanks for the review. Updated diff attached.<br>
<br>
Cheers,<br>
<br>
James<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D13551" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13551</a><br>
<br>
Files:<br>
  include/llvm/Analysis/VectorUtils.h<br>
  lib/Analysis/VectorUtils.cpp<br>
  lib/Transforms/Vectorize/LoopVectorize.cpp<br>
  test/Transforms/LoopVectorize/AArch64/loop-vectorization-factors.ll<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>