On Fri, Apr 11, 2014 at 04:57:42PM -0700, Jerry DeLisle wrote: > On 04/11/2014 04:31 PM, Tobias Burnus wrote: > > Jerry DeLisle wrote: > >> I plan to commit the following patch to 4.10, 4.9, 4.8, and 4.7. > >> > >> It is a partial revert of the patch to PR38199 which is an optimization of > >> internal unit reads. > >> > >> The original patch was too aggressive. After this revert, I will > >> investigate > >> further to see if I can refine it further. > > > > This partial revert looks fine to me - too bad that we have only now learned > > about this problem. > > > > For 4.9 prior the 4.9.0 release, you need approval by Jakub, who probably > > would > > like to avoid to have another RC. In case that there is a second RC, I > > think it > > would make sense to include this patch as well. > > > > Tobias > > > > I am waiting for Jakub's approval for 4.9.
Given that this is a regression from 4.8.2 and only went into the release branches after that, I think it is ok for 4.9.0 right now, but please make sure it is tested sufficiently (beyond make check, try some larger Fortran apps). Jakub