This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [testsuite, i386] Fix for PR50185

From: Uros Bizjak <ubizjak at gmail dot com>
To: Kirill Yukhin <kirill dot yukhin at gmail dot com>
Cc: gcc-patches List <gcc-patches at gcc dot gnu dot org>, "H.J. Lu" <hjl dot tools at gmail dot com>, Jakub Jelinek <jakub at redhat dot com>
Date: Fri, 26 Aug 2011 14:46:14 +0200
Subject: Re: [testsuite, i386] Fix for PR50185
References: <CAGs3RftShBU4apxfTS2bEbS4epcvFSMur32msHXV52KJ_kdijg@mail.gmail.com> <CAGs3RfsRaNYYJAKWw7oH0oEmqpz8cbSb70yXjhW0b-527zSrWg@mail.gmail.com>

On Fri, Aug 26, 2011 at 2:04 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
> According to Jakub's input, I've updated test to scan instruction, not
> pattern name.
>
> Is it ok?
>
> Thanks, K
>
> On Fri, Aug 26, 2011 at 3:45 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
>> Hi,
>> Here is a fix for http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50182
>>
>> testsuite/ChangeLog entry:
>>
>> 2011-08-26 ?Kirill Yukhin ?<kirill.yukhin@intel.com>
>>
>> ? ? ? ?PR testsuite/50185
>> ? ? ? ?* gcc.target/i386/avx2-vmovmskb-2.c: Rename to ...
>> ? ? ? ?* gcc.target/i386/avx2-vpmovmskb-2.c: ... this. Update.
>>
>> Test passes.
>> Ok for trunk?

Is this correct ChangeLog? Looking into the patch, you are changing
one test to look for insn name, while adding avx2-vpmovmskb-2.c which
still looks for pattern name.

Please update ChangeLog and/or attached patch.

Uros.

References:
- [testsuite, i386] Fix for PR50185
  - From: Kirill Yukhin
- Re: [testsuite, i386] Fix for PR50185
  - From: Kirill Yukhin

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]