This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
*PING* Re: [Patch, Fortran] PR 43331 - Fix Cray pointer (middle-end) type declaration
- From: Tobias Burnus <burnus at net-b dot de>
- To: gcc patches <gcc-patches at gcc dot gnu dot org>, gfortran <fortran at gcc dot gnu dot org>
- Date: Wed, 17 Mar 2010 07:51:13 +0100
- Subject: *PING* Re: [Patch, Fortran] PR 43331 - Fix Cray pointer (middle-end) type declaration
- References: <4B99487C.4070507@net-b.de>
*PING*
I am pinging for the combination of the following two patches:
http://gcc.gnu.org/ml/fortran/2010-03/msg00061.html
http://gcc.gnu.org/ml/fortran/2010-03/msg00062.html
Asher was so kind to run their programs with those patches: "the patch
is an improvement, since it removes some unneeded complexity: I never
liked having that artificial constant bound! I tested the changes on a
few codes, and it seems fine."
OK for the trunk?
Tobias
On 11 March 2010 20:46, Tobias Burnus wrote:
> For some reason, assumed-size Cray pointees had an upper bound of one,
> which causes problems with middle-end optimizations (see PR).
>
> This patch changes it have an unknown upper bound. The patch is straight
> forward, except for the change in trans-decl.c's
> gfc_trans_deferred_vars, where I simply ignore Cray pointees. I am not
> sure whether this is correct, but it seems to work and I cannot imagine
> initializations to Cray pointees for which one would need to pass the
> symbol to gfc_trans_g77_array. If one does the call, it ICEs due to a
> NULL pointer.
>
> Build and regtested on x86-64-linux. OK for the trunk?
>
> Tobias
>
> PS: I have not added a test case as - unpatched - the
> cray_pointers_2.f90 test would fail as soon as the middle-end change is in.
>