Patchwork Fix testcase for pr44391 to run only for x86

login
register
mail settings
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

Ramana Radhakrishnan - July 7, 2010, 10:08 p.m.
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.

 {
Sebastian Pop - July 7, 2010, 10:11 p.m.
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)