gcc-patches-ow...@gcc.gnu.org wrote on 12/12/2011 01:00:52 PM:
> I changed xfails to target-checks - for now I use common > vect_multiple_sizes (though it'll fail when wider vectors emerge). > Also, I changed AVX-check to the version Uros suggested. Please check > updated patch (attached). > > As for vect_multiple_sizes_32B_16B and similar - isn't it too > target-specific? I think if we want to keep everything as general as > possible, we should have something like vect_1_vector_size_available, > vect_2_vector_sizes_available, etc. I think there is a difference between different vector sizes, and calling it vect_X_vector_size_available is not sufficient. Your patch will cause failures on ARM. It has two vector sizes, 16 and 8 bytes. E.g., vect-33.c gets vectorized with the default vector size, and the alignment message should be printed only once, and not twice as with your patch. So, it looks like you need several vect_multiple_sizes_X. Ira
