On Sat, Oct 8, 2011 at 3:56 PM, H.J. Lu <hjl.to...@gmail.com> wrote: > On Fri, Oct 7, 2011 at 3:49 PM, Richard Henderson <r...@redhat.com> wrote: >> Our existing vector permutation support uses the name vec_perm. >> The new support added for __builtin_shuffle used a variety of >> names. This brings all the internals into line with vec_perm. >> >> I've added documentation for the (existing) variable permute >> rtl named pattern (vec_perm). I've also added documentation for >> the (intended) constant permute rtl named pattern (vec_perm_const). >> I've not actually added the support for vec_perm_const yet, but >> plan to do so shortly. It was just easiest to do the docs for >> both at the same time. >> >> The builtin is still named shuffle in honor of the OpenCL primitive >> on which this feature is based. I've improved the documentation >> for the builtin a bit to make it clear that the selector modulus >> is part of the interface. >> >> Tested on x86_64-linux. >> > > Is this a typo: > > @@ -557,6 +560,7 @@ enum optab_index > #define vec_pack_usat_optab (&optab_table[OTI_vec_pack_usat]) > #define vec_pack_sfix_trunc_optab (&optab_table[OTI_vec_pack_sfix_trunc]) > #define vec_pack_ufix_trunc_optab (&optab_table[OTI_vec_pack_ufix_trunc]) > +#define vec_perm_optab (&direct_optab_table[(int) OTI_vec_perm]) > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > #define powi_optab (&optab_table[OTI_powi]) >
DOI_vshuffle is renamed/moved to OTI_vec_perm. But the rename/move is incomplete, which caused: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50665 -- H.J.
