[PATCH] Fix ix86_expand_vector_set (PR target/70421)

Jakub Jelinek jakub@redhat.com
Wed Mar 30 14:13:00 GMT 2016


On Wed, Mar 30, 2016 at 04:53:48PM +0300, Kirill Yukhin wrote:
> I think that order of arguments in built-in might be changed easily.
> This doesn't affect intrinsics at all, because of that.
> 
> I can hardly recall, but my bet is that this order was dictated by:
> ix86_expand_sse_movcc where order of blends args should corresond
> on AVX*. 

Having the AVX512* blends have different order from AVX{,2} blends would be
bad though, so if we want to change the order, we'd have to change it
everywhere.

> Jakub, nay be add comment in the patch on blendm patterns emphasizing
> this non-regular order?

Ok, will do.

	Jakub



More information about the Gcc-patches mailing list