This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][PING] PR62120
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Ilya Tocar <tocarip dot intel at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 30 Sep 2014 12:49:02 +0200
- Subject: Re: [PATCH][PING] PR62120
- Authentication-results: sourceware.org; auth=none
- References: <20140901104314 dot GB26266 at msticlxl7 dot ims dot intel dot com> <20140901105552 dot GN17454 at tucnak dot redhat dot com> <20140901143816 dot GC26266 at msticlxl7 dot ims dot intel dot com> <20140915144327 dot GB1778 at msticlxl7 dot ims dot intel dot com> <20140930104421 dot GA20054 at msticlxl7 dot ims dot intel dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Sep 30, 2014 at 02:44:21PM +0400, Ilya Tocar wrote:
> > 2014-09-15 Ilya Tocar <ilya.tocar@intel.com>
> >
> > PR middle-end/62120
> > * varasm.c (decode_reg_name_and_count): Check availability for
> > registers from ADDITIONAL_REGISTER_NAMES.
> >
> > Testsuite/
> >
> > 2014-09-15 Ilya Tocar <ilya.tocar@intel.com>
> >
> > PR middle-end/62120
> > * gcc.target/i386/avx512f-additional-reg-names.c: Use register vaild
s/vaild/valid/
> > in 32-bit mode.
> > * gcc.target/i386/pr62120.c: New.
> >
> > ---
> > gcc/testsuite/gcc.target/i386/avx512f-additional-reg-names.c | 2 +-
> > gcc/testsuite/gcc.target/i386/pr62120.c | 8 ++++++++
> > gcc/varasm.c | 5 +++--
> > 3 files changed, 12 insertions(+), 3 deletions(-)
> > create mode 100644 gcc/testsuite/gcc.target/i386/pr62120.c
> >
> > diff --git a/gcc/testsuite/gcc.target/i386/avx512f-additional-reg-names.c b/gcc/testsuite/gcc.target/i386/avx512f-additional-reg-names.c
> > index 164a1de..98a9052 100644
> > --- a/gcc/testsuite/gcc.target/i386/avx512f-additional-reg-names.c
> > +++ b/gcc/testsuite/gcc.target/i386/avx512f-additional-reg-names.c
> > @@ -3,7 +3,7 @@
> >
> > void foo ()
> > {
> > - register int zmm_var asm ("zmm9") __attribute__((unused));
> > + register int zmm_var asm ("zmm7") __attribute__((unused));
> >
> > __asm__ __volatile__("vxorpd %%zmm0, %%zmm0, %%zmm7\n" : : : "zmm7" );
Please use zmm6 instead, zmm7 is clobbered in the following statement.
Otherwise LGTM.
Jakub