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: [Fortran, Patch] PR31188 - Allow array dummy as range for constant arrays

From: FX Coudert <fxcoudert at gmail dot com>
To: Tobias Burnus <burnus at net-b dot de>
Cc: "'fortran at gcc dot gnu dot org'" <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
Date: Thu, 15 Mar 2007 22:33:51 +0100
Subject: Re: [Fortran, Patch] PR31188 - Allow array dummy as range for constant arrays
References: <45F9B189.1020608@net-b.de>

2007-03-15 Tobias Burnus <burnus@net-b.de>

	PR fortran/31188
	* expr.c (gfc_simplify_expr,find_array_section): Support
	  non-expression-constant variables.

Paul proposed another patch in the PR trail, which looks cleaner to me as it doesn't add the expr_const arg. Now I imagine there was a reason for you adding it in the first place, could you comment on Paul's patch (or could Paul comment on your patch)?

Otherwise, the reasoning looks OK, and I think when you both agree, you can go ahead and commit whatever it is you decided. (and, if both patches are equivalent, I'd prefer we don't add the expr_const argument).

Thanks for acting so fast,
FX

Follow-Ups:
- Re: [Fortran, Patch] PR31188 - Allow array dummy as range for constant arrays
  - From: Tobias Burnus

References:
- [Fortran, Patch] PR31188 - Allow array dummy as range for constant arrays
  - From: Tobias Burnus

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