This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH i386 AVX512] [8/n] Extend substs for new patterns.
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Kirill Yukhin <kirill dot yukhin at gmail dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Richard Henderson <rth at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 12 Aug 2014 20:16:31 +0200
- Subject: Re: [PATCH i386 AVX512] [8/n] Extend substs for new patterns.
- Authentication-results: sourceware.org; auth=none
- References: <20140812125804 dot GB916 at msticlxl57 dot ims dot intel dot com>
On Tue, Aug 12, 2014 at 2:58 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
> Hello,
> This patch extends substs/subst_attrs to be used
> with new patterns.
>
> Bootstrapped.
>
> Is it ok for trunk?
In a couple of places, <MODE_SIZE> checks are changed to GET_MODE_SIZE
(GET_MODE (operands[0])) and in a similar way, <MODE>mode checks were
changed to GET_MODE (operands[0]). The previous checks are more
efficient and are preferred. Is there a reason you need to check
operands[0] dynamically?
Uros.