This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH,fortran]: partial fix for PR 32600
- From: Tobias Burnus <burnus at net-b dot de>
- To: Tobias Schlüter <tobias dot schlueter at physik dot uni-muenchen dot de>
- Cc: "Christopher D. Rickett" <crickett at lanl dot gov>, fortran at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Mon, 15 Oct 2007 18:31:45 +0200
- Subject: Re: [PATCH,fortran]: partial fix for PR 32600
- References: <Pine.LNX.4.64.0710121357480.20659@lennox.lanl.gov> <47136FFB.8000605@physik.uni-muenchen.de>
Tobias Schlüter wrote:
> I think the patch is ok, but I would appreciate if one of our
> ISO-C-bindings experts gave it a second look.
I also think it is ok.
Christopher D. Rickett wrote:
> there are two main parts to the patch; one is mostly in trans-expr.c
> and it generates the inlined code and the other is simply removing
> c_associated from libgfortran. the part for the frontend does not
> depend on the changes to libgfortran, so it is compatible with
> existing versions of the library for gcc 4.3.0 if it is too late to
> change the library.
Tobias Schlüter wrote:
> I agree that this is too late for 4.3, as the enhancement is barely
> user-visible.
I actually would like if we could still get it into 4.3.0 as this will
remove dead library code and the change is very localized and not a new
feature; however, it is strictly speaking also not a bug fix (but a
clean up/dead code removal/optimization improvement).
How is your OK meant: OK for 4.4.x (implies w/o library change for
compatible reasons), OK for 4.3.x (w/o library change) or OK for 4.3.0
w/ the library change?
Tobias