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

Reply via email to