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] PR40011 - Problems with -fwhole-file


On 05/24/2010 10:10 PM, Paul Richard Thomas wrote:
> I would now like proposals about how to introduce -fwhole-file as the default:
> (i) A -fno-whole-file or would std=legacy as an over-ride?

I am in favour of (keeping) -fno-whole-file.

In the long run, I would like to see -fno-whole-file deprecated. I
recall correctly, not using this option can yield wrong code in some
cases (I think there is a PR somewhere) - and Steven said that the
middle end contains some code which deals which some cleanup which is
otherwise not needed. If a code fails to compile due to wrong
declarations, we might consider adding an override option - acting like
-fno-whole-file for the user, but working differently internally. In any
case, we need at least one full release before one can remove the option
to get good user-code coverage.


> Bootstrapped and regtested on x86_64/FC9 - OK for trunk?

OK. Thanks for the patch.

Tobias

> 2010-05-24  Paul Thomas  <pault@gcc.gnu.org>
> 
> 	PR fortran/40011
> 	* resolve.c (resolve_global_procedure): Resolve the gsymbol's
> 	namespace before trying to reorder the gsymbols.
> 
> 2010-05-24  Paul Thomas  <pault@gcc.gnu.org>
> 
> 	PR fortran/40011
> 	* gfortran.dg/whole_file_18.f90 : New test.


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