[PATCH,fortran]: partial fix for PR 32600

Tobias Burnus burnus@net-b.de
Mon Oct 15 17:08:00 GMT 2007


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



More information about the Gcc-patches mailing list