Re: [patch, Fortran] Fix PR 71902

2016-08-21 Thread Mikael Morin
Le 18/08/2016 à 21:24, Thomas Koenig a écrit : Hello Mikael, This doesn't look correct, what about substrings following component or array references? You're right; corrected in the attached patch. I have also added a test case for this. PS: What about the original fix, wasn't it a depende

Re: [patch, Fortran] Fix PR 71902

2016-08-18 Thread Thomas Koenig
Hello Mikael, This doesn't look correct, what about substrings following component or array references? You're right; corrected in the attached patch. I have also added a test case for this. PS: What about the original fix, wasn't it a dependency problem/inaccuracy after all? The main err

Re: [patch, Fortran] Fix PR 71902

2016-08-16 Thread Mikael Morin
Hello, Le 14/08/2016 à 15:05, Thomas Koenig a écrit : Index: frontend-passes.c === --- frontend-passes.c (Revision 239218) +++ frontend-passes.c (Arbeitskopie) @@ -177,6 +178,13 @@ realloc_string_callback (gfc_code **c, int *wal

[patch, Fortran] Fix PR 71902

2016-08-14 Thread Thomas Koenig
Hello world, this patch fixes the performance regression introduced with the fix for 71783. It also corrects an error in the logic in realloc_string_callback by now checking if the experssion is deferred, instead of checking for the symbol having the allocatable attribute (which led to false pos