Re: [Patch, Fortran] Fix PR 50564

2011-10-10 Thread Thomas Koenig
Hi Tobias, In conclusion: I am fine with the FORALL part, but not with the DO CONCURRENT part. Yep, you're right. I have modified the patch accordingly. Here is what I committed. Best regards Thomas Index: fortran/ChangeLog ==

Re: [Patch, Fortran] Fix PR 50564

2011-10-10 Thread Tobias Burnus
Hi Thomas, > the attached patch fixes the PR by removing common function elimination > in FORALL statements. I think that part is OK. However, you do more - you also avoid the handling for DO CONCURRENT. However, I do not see why it could fail for DO CONCURRENT or rather: I do not see a case whe

[Patch, Fortran] Fix PR 50564

2011-10-09 Thread Thomas Koenig
Hello world, the attached patch fixes the PR by removing common function elimination in FORALL statements. In the course of fixing this PR, I had originally fixed the ICE only to find that the transformation (where f is a function) forall (i=1:2) a(i) = f(i) + f(i) end forall to forall (i=1