On Sun, Dec 22, 2013 at 11:47:52AM +0100, Uros Bizjak wrote:
> * tree-vect-stmts.c (vectorizable_load): Support AVX512's gathers.
> * tree-vectorizer.h (MAX_VECTORIZATION_FACTOR): Extend for 512
> bit vectors.
>
> I assumed the same testing procedure as described in the original submission:
>
> Testing:
> 1. Bootstrap pass.
> 2. make check shows no regressions.
> 3. Spec 2000 & 2006 build show no regressions both with and without
> -mavx512f option.
> 4. Spec 2000 & 2006 run shows no stability regressions without
> -mavx512f option.
>
> The x86 part is OK for mainline. You will also need approval from the
> middle-end reviewer for tree-* parts.
The tree parts are ok for trunk, but likely insufficient by now,
you need similar changes for vectorizable_mask_load_store (which also
handles gathers), plus verify even the non-gather mask load/store if they
don't need any tweaking for AVX512F (integer masks rather than vector
ones?).
Jakub
