[PATCH] D8943: Calculate vectorization factor using the narrowest type instead of widest type

Cong Hou via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 23 16:27:17 PDT 2015


congh added a comment.

In http://reviews.llvm.org/D8943#252089, @hfinkel wrote:

> Have you run LLVM's test suite with this turned on? Are there any significant performance changes? [I'm happy for this to go in, given that it's disabled by default, even if there are regressions to fix, but I'd like to know where we stand].
>
> .


I ran the regression tests and only three failed, one of which (assertion fail) is already fixed in the updated patch, and the other two are caused by larger VF and are understood.

The performance test is still running and I will give the result later.


http://reviews.llvm.org/D8943





More information about the llvm-commits mailing list