This is the mail archive of the 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: [PATCH] Fix new split-1.c testcase

On Sun, Jun 11, 2017 at 07:38:04PM -0700, Ian Lance Taylor wrote:
> On Sun, Jun 11, 2017 at 4:40 AM, Segher Boessenkool
> <> wrote:
> >
> > The new split-1.c testcase fails on targets that do not support split
> > stack (like 32-bit PowerPC Linux).  This patch fixes it by only running
> > the testcase if split stack is supported.  It also adds the reorder
> > flag to the options, so that the test actually tests what it says it
> > tests.
> >
> > Is this okay for trunk?
> Whoops, sorry about that.
> Adding dg-require-effective-target split_stack is fine.  Adding an
> explicit -freorder-blocks-and-partition option is not.  Adding the
> explicit option will cause the test to fail when using gold, as the
> two options are not compatible.  The point of the test is to test that
> using -fsplit-stack disables the default enabling of
> -freorder-blocks-and-partition.

Ah, I see.  Could you change the comment then, to say what we are
really testing?

> >  /* Test case that we don't get a link-time error when using
> >     -fsplit-stack with -freorder-blocks-and-partition.  */

> >  /* { dg-require-effective-target freorder } */

And this line isn't required, in that case -- removing it is less
confusing and allows the test to run in more places ;-)

[ Paul Hua sent a patch adding split_stack already, it was OKed, but
it is not committed yet, fwiw ].


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