Re: [PATCH, fortran] Add -frecursive and fix local array handling with -fopenmp

[Jakub Jelinek <jakub@redhat.com>]

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.

Silently changing RLIMIT_STACK in libgfortran is IMHO not a good idea,
for OpenMP programs that will not help, because at that point libpthread
is already initialized and thread stacks will have different default size
already anyway, and when Fortran programs spawn other programs it would
cause bad havoc (e.g. recent make decided to do this silly thing and
suddenly several programs started weirdly misbehaving when run from
Makefile as opposed to when run without make - we had to change make
to make sure stack limit is restored before execve).

	Jakub