On Mon, 2018-06-04 at 10:04 -0700, Matt Turner wrote: > #ifdef HAVE_GCC_VECTOR_EXTENSIONS > - const uint8x16 bswap_shufflemask = > +# if __has_builtin(__builtin_shufflevector) > + randdata.vb = > + __builtin_shufflevector (randdata.vb, randdata.vb, > + 3, 2, 1, 0, 7, 6 , 5, 4, > + 11, 10, 9, 8, 15, 14, 13, 12); > +# else > + static const uint8x16 bswap_shufflemask = ^ ^^^^^^
Seems superfluous, though I guess it doesn't change semantics. With or without that bit: Reviewed-by: Adam Jackson <a...@redhat.com> I think we're starting to be well overdue for an 0.36 release, but I'd like to take the opportunity to suggest moving to fdo's gitlab as we do that. I already have a copy imported personally and have CI working: https://gitlab.freedesktop.org/ajax/pixman/-/jobs/986 - ajax _______________________________________________ Pixman mailing list Pixman@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/pixman