This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, fortran] Add -frecursive and fix local array handling with -fopenmp
On 8/16/07, Jakub Jelinek <jakub@redhat.com> wrote:
> On Thu, Aug 16, 2007 at 10:50:52AM +0200, Tobias Burnus wrote:
> > Note that you only see "weird things" if you have either parallel code
> > (-fopenmp) which is fixed by Asher's patch or whether you are using
> > recurse code without specifying RESCURSIVE, which is invalid Fortran.
> >
> >
> > Regarding Asher's patch, I think it is OK as it.
>
> I don't like that, overriding explicit -fmax-stack-var-size=XXX by -fopenmp
> sounds like a very bad idea to me.
> -fopenmp certainly can change the default, when -fmax-stack-var-size=XXX
> isn't explicitly used, from 32K to -1, but if the user requests particular
> cutoff we should honor that.
That's reasonable. The ifort compiler seems to do the same thing, but
issues a warning:
% ifort -save -openmp openmp_stack.f90
ifort: warning: explicit static allocation of locals specified,
overriding OpenMP's implicit auto allocation
When -recursive and -save are specified together, ifort seems to
silently override the -save flag.
-Asher