Re: [Patch, Fortran] Fix previous patch

2015-01-08 Thread Tobias Burnus
Paul Richard Thomas wrote: It looks to me as if you had to do a fair amount of detective work there! The patch is OK for trunk. Thanks for the review - which didn't make it to mailing list for some reasons (text+HTML email?). Committed as Rev. 219354 Tobias Thanks for your efforts Paul

Re: [Patch, Fortran] Fix previous patch

2015-01-07 Thread Tobias Burnus
Early PING: https://gcc.gnu.org/ml/gcc-patches/2015-01/msg00090.html Tobias Burnus wrote: Attached is a regtested patch, which fixes the issue. Additionally, the variable visibility (TREE_PUBLIC) is now depending on the private attribute (copied from the module var generation) and I mark the tr

Re: [Patch, Fortran] Fix previous patch (was: Re: [Patch, Fortran + Testsuite] Fix coarray handling in modules)

2015-01-04 Thread Dominique d'Humières
Dear Tobias, I have done a clean bootstrap with your patch and run make -k check-gfortran RUNTESTFLAGS="caf.exp --target_board=unix'{-m32,-m64}’" without regression. Thanks, Dominique > Le 4 janv. 2015 à 19:57, Tobias Burnus a écrit : > > Attached is a regtested patch, which fixes the issu

[Patch, Fortran] Fix previous patch (was: Re: [Patch, Fortran + Testsuite] Fix coarray handling in modules)

2015-01-04 Thread Tobias Burnus
Attached is a regtested patch, which fixes the issue. Additionally, the variable visibility (TREE_PUBLIC) is now depending on the private attribute (copied from the module var generation) and I mark the tree as DECL_NONALIASED. The former I also did for proc-pointers, which is an unrelated patc