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: RFA: Fix isl-ast-gen-if-1.c test

From: Jeff Law <law at redhat dot com>
To: Nicholas Clifton <nickc at redhat dot com>, gcc-patches at gcc dot gnu dot org
Date: Mon, 22 Jun 2015 10:17:31 -0600
Subject: Re: RFA: Fix isl-ast-gen-if-1.c test
Authentication-results: sourceware.org; auth=none
References: <87r3p3sqh6 dot fsf at redhat dot com> <55882C9C dot 9060405 at redhat dot com> <558832C1 dot 4030109 at redhat dot com>

On 06/22/2015 10:07 AM, Nicholas Clifton wrote:

Hi Jeff,

I'd tend to prefer to change the size of the array -- adding another
conditional in the loop may have unintended consequences that possibly
scramble things just enough to compromise the test.


Okey dokey, here is a revised version.  Is this one OK ?

Cheers
   Nick

gcc/ChangeLog

Index: 2015-06-22  Nick Clifton  <nickc@redhat.com>

     * gcc.dg/graphite/isl-ast-gen-if.c (main): Increase size of a
     array to allow a[50] to be a valid location.

OK.
jeff

References:
- RFA: Fix isl-ast-gen-if-1.c test
  - From: Nick Clifton
- Re: RFA: Fix isl-ast-gen-if-1.c test
  - From: Jeff Law
- Re: RFA: Fix isl-ast-gen-if-1.c test
  - From: Nicholas Clifton

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