This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, i386, AVX-512] Fix splitter for `not-xor' logic.
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Kirill Yukhin <kirill dot yukhin at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 21 Sep 2015 14:37:05 +0200
- Subject: Re: [PATCH, i386, AVX-512] Fix splitter for `not-xor' logic.
- Authentication-results: sourceware.org; auth=none
- References: <20150921105002 dot GB49423 at msticlxl57 dot ims dot intel dot com>
On Mon, Sep 21, 2015 at 12:50 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
> Hello UroÅ,
>
> This simple patch fixes mode (to iterator) in
> splitter for `not-xor' logic.
>
> Bootstrapped & reg-tested on trunk.
>
> Is it ok for trunk?
>
> Is it ok for gcc-5-branch? (if tested accordingly)
OK everywhere. Looks obvious, though.
> (I have no idea how to reference to define_split in
> ChangeLog entry)
Yes, you need a bit of creativity to describe define_splits, but looks OK to me.
>
> gcc/
> * gcc/config/i386/i386.md (define_split not/xor SWI1248x): Use
> iterator instead of fixed modes..
Thanks,
Uros.