On Tue, Apr 8, 2014 at 12:07 AM, Jakub Jelinek <ja...@redhat.com> wrote:
> On Mon, Apr 07, 2014 at 12:16:12PM -0700, Cong Hou wrote:
>> --- a/gcc/testsuite/ChangeLog
>> +++ b/gcc/testsuite/ChangeLog
>> @@ -1,3 +1,11 @@
>> +2014-04-07 Cong Hou <co...@google.com>
>> +
>> + PR testsuite/60773
>> + * testsuite/lib/target-supports.exp:
>> + Add check_effective_target_vect_widen_mult_si_to_di_pattern.
>
> No testsuite/ prefix here. Please write it as:
> * lib/target-supports.exp
> (check_effective_target_vect_widen_si_to_di_pattern): New.
Thank you for pointing it out. Corrected.
>
>> --- a/gcc/testsuite/gcc.dg/vect/pr60656.c
>> +++ b/gcc/testsuite/gcc.dg/vect/pr60656.c
>> @@ -1,5 +1,7 @@
>> /* { dg-require-effective-target vect_int } */
>> +/* { dg-require-effective-target vect_long } */
>>
>> +#include <stdarg.h>
>
> I fail to see why you need this include, neither your test nor tree-vect.h
> uses va_*.
I have removed this include.
thanks,
Cong
>
> Otherwise looks good to me.
>
> Jakub
