[testsuite]MIPS remove duplicate div-x test

Matthew Fortune Matthew.Fortune@imgtec.com
Sun Jun 4 20:41:00 GMT 2017


Hi Paul,

Paul Hua <paul.hua.gm@gmail.com> writes:
> cc: Matthew.
> 
> ping.

Sorry a little slow on the reply.
 
> On Thu, Jun 1, 2017 at 3:35 PM, Paul Hua <paul.hua.gm@gmail.com> wrote:
> > Hi,
> >
> > There are duplicate testcase in gcc.target/mips dir.
> >
> > div-5.c same as div-9.c.
> > div-6.c same as div-10.c.
> > div-7.c same as div-11.c.
> > div-8.c same as div-12.c.
> >
> > Is this deliberate?

I see no evidence of this being deliberate and has been like this since
the original commit.

> > Otherwise, the attached patch fixing this.
> >
> >
> > Paul.
> >
> > ***ChangeLog***
> >
> > 2017-06-01    Chenghua Xu <paul.hua.gm@gmail.com>
> >
> >     Remove duplicate div-x testcase.

These kind of comments don't tend to go in a changelog.

> >     * gcc.target/mips/div-9.c: Delete.

You could say "Delete duplicate test" here if you want though.

> >     * gcc.target/mips/div-10.c: Ditto.
> >     * gcc.target/mips/div-11.c: Ditto.
> >     * gcc.target/mips/div-12.c: Ditto.

Otherwise OK. I can't remember if you have write access let me know
if you need it committing. Thanks for finding this.

Matthew


More information about the Gcc-patches mailing list