| Submitter | Ramana Radhakrishnan |
|---|---|
| Date | July 7, 2010, 10:08 p.m. |
| Message ID | <AANLkTin1KqYBLyHVkqrPo9lnnqDCvBf1ELk8CbJLuL98@mail.gmail.com> |
| Download | mbox | patch |
| Permalink | /patch/58200/ |
| State | New |
| Headers | show |
Comments
On Wed, Jul 7, 2010 at 17:08, Ramana Radhakrishnan <ramana.gcc@googlemail.com> wrote: > Hi, > > This fixes an unnecessary failure for pr44391.c on any platform that > doesn't support the -m32 command line option. I noticed this failing > on the ARM port because there doesn't exist a -m32 option to the ARM > backend and presumably this test was targeted only for the x86 port. > > I'm not very familiar with the idioms for the x86 port or indeed the > variations in the testsuite so additional eyeballs would be > appreciated. > > Ok to commit on trunk? > Ok. Thanks, Sebastian Pop -- AMD / Open Source Compiler Engineering / GNU Tools
Patch
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 } } } */ void byte_insert_op1 (unsigned char *loc, unsigned char *end, unsigned *pto)
Hi, This fixes an unnecessary failure for pr44391.c on any platform that doesn't support the -m32 command line option. I noticed this failing on the ARM port because there doesn't exist a -m32 option to the ARM backend and presumably this test was targeted only for the x86 port. I'm not very familiar with the idioms for the x86 port or indeed the variations in the testsuite so additional eyeballs would be appreciated. Ok to commit on trunk? Thanks, Ramana 2010-07-07 Ramana Radhakrishnan <ramana.radhakrishnan@arm.com> * gcc.dg/graphite/pr44391.c: Run only for x86 platforms. {