On Tue, Apr 08, 2014 at 07:40:09PM -0700, Cong Hou wrote: > 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.
Only partially, I see there just the #include removed and testsuite/ prefix removed, but not the using the style where in () is the name of function that has been added/modified and after ): what has been done with it. Anyway, please commit it with the above proposed change, this is a P1 and thus urgent. Thanks. Jakub