On Wed, Nov 2, 2011 at 09:33, Tobias Burnus <bur...@net-b.de> wrote: > Dear all, > > attached is an updated version of Patch 2. The change is that I removed the > global variable for fill_st_vector and updated the comment for > do_traverse_symtree to make assumptions clearer. > > This version of the patch was build and regtested (gfortran + libgomp). > OK?
Ok. It seems a bit unclear whether marking helps or not, but at least it doesn't seem to make anything worse.. > > > Dear Tobias S., > > Tobias Schlüter wrote: >> >> I don't remember all this very clearly, but I think that the >> gfc_symbol::tlink field is intended for something like this > > I think that's a rather different issue: It relates to undoing a symbol, > which I do not want to do. Additionally, a tlink symbol is already added to > a symtree (which is hence rebalanced). One possibility is to delay the > inclusion of symbols into the symtree - and do only so after the traversal - > but that might cause issues if one later searches in called function for > that symtree. Thus, I think my current patch should be sufficiently simple, > fast and solid. > > Tobias > -- Janne Blomqvist
