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, fortran] PR31154 , PR31229 and PR33334 - use associated or imported function characteristics.

From: Jerry DeLisle <jvdelisle at verizon dot net>
To: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
Cc: Dominique Dhumieres <dominiq at lps dot ens dot fr>, gcc-patches List <gcc-patches at gcc dot gnu dot org>, "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>
Date: Mon, 01 Oct 2007 21:53:18 -0700
Subject: Re: [Patch, fortran] PR31154 , PR31229 and PR33334 - use associated or imported function characteristics.
References: <20070926152517.634285BB9F@mailhost.lps.ens.fr> <339c37f20709290257u5424ebffrddc61a9302b58958@mail.gmail.com> <339c37f20709290859q19794ac3hc7ef7ef8d24f540d@mail.gmail.com>

Paul Richard Thomas wrote:

This I believe it it:

(i) Have added ST_DERIVED_DECL to the conditions where a check is made
to see if the derived type is available.
(ii) I have corrected the illegailities in derived_function_interface_1.f90.

With this, the patch bootstraps and regtests on x86_ia64/FC5 (no, I
did not miss anything this time!), CP2K compiles and runs, as does
tonto.

OK for trunk?

Paul

Paul,

Typo in comment: spelling of "there"


+   /* If we still have kind = -1 at the end of the specification block,
+      then ther is an error. */

Also: Is gfc_current_block() a macro? If not, need space before the left paren in several places.

Otherwise OK to commit.

Regards,

Jerry

Follow-Ups:
- Re: [Patch, fortran] PR31154 , PR31229 and PR33334 - use associated or imported function characteristics.
  - From: Steve Kargl

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