This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix ix86_expand_vector_set (PR target/70421)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Kirill Yukhin <kirill dot yukhin at gmail dot com>
- Cc: Jeff Law <law at redhat dot com>, Uros Bizjak <ubizjak at gmail dot com>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 30 Mar 2016 15:59:46 +0200
- Subject: Re: [PATCH] Fix ix86_expand_vector_set (PR target/70421)
- Authentication-results: sourceware.org; auth=none
- References: <20160329170521 dot GM3017 at tucnak dot redhat dot com> <56FABEEF dot 5030400 at redhat dot com> <20160329174920 dot GS3017 at tucnak dot redhat dot com> <20160330135346 dot GA58184 at msticlxl57 dot ims dot intel dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
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