This is the mail archive of the gcc-patches@gcc.gnu.org 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 testcase for pr44391 to run only for x86


On Thu, Jul 08, 2010 at 11:18:43AM +0200, Richard Guenther wrote:
> On Thu, Jul 8, 2010 at 8:36 AM, Uros Bizjak <ubizjak@gmail.com> wrote:
> >> diff --git a/gcc/testsuite/gcc.dg/graphite/pr44391.c
> >> b/gcc/testsuite/gcc.dg/graphite/pr44391.c
> >> index 91b3afa..dde2fb7 100644
> >> --- a/gcc/testsuite/gcc.dg/graphite/pr44391.c
> >> +++ b/gcc/testsuite/gcc.dg/graphite/pr44391.c
> >> @@ -1,4 +1,4 @@
> >> -/* { dg-options "-Os -m32 -fgraphite-identity -ffast-math" } */
> >> +/* { dg-options "-Os -m32 -fgraphite-identity -ffast-math" ?{ target
> >> { { i?86-*-* x86_64-*-* } && ilp32 } } } */
> >
> > You should just remove -m32 from the options string. There is nothing
> > x86 or 32bit specific in the test.
> 
> Or it should use { dg-effective-target ilp32 }.

I guess not all ilp32 targets support -m32 option.  Anyway, Uros' suggestion
seems to be my preference too.

	Jakub


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