[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-11-23 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #27 from Jerry DeLisle --- This should be fixed now fairly well. At this point there are not a lot of major issues in our own libgfortran library. I will leave this bug report open for a while if any issues arise. Here is my latest

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-11-23 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #26 from Jerry DeLisle --- Author: jvdelisle Date: Thu Nov 23 17:19:18 2017 New Revision: 255108 URL: https://gcc.gnu.org/viewcvs?rev=255108=gcc=rev Log: 2017-11-23 Jerry DeLisle Backport from

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-11-20 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #25 from Jerry DeLisle --- Author: jvdelisle Date: Tue Nov 21 02:17:11 2017 New Revision: 254982 URL: https://gcc.gnu.org/viewcvs?rev=254982=gcc=rev Log: 2017-11-20 Jerry DeLisle PR

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-11-18 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #24 from Jerry DeLisle --- After a minor test tweak. $ gfc -static pr78549.f $ time ./a.out real0m24.049s user0m22.941s sys 0m0.962s $ gfc6 -static pr78549.f $ time ./a.out real0m22.838s user0m22.718s sys

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-11-16 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #23 from Jerry DeLisle --- Well, instrumenting a little bit I see that delete_root is getting called many many many times. So, every call to newunit_alloc is assigning a new unit number which is getting added to the treap and never

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-11-10 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 Thomas Koenig changed: What|Removed |Added CC||manfred99 at gmx dot ch --- Comment #22

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-10-07 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #21 from Thomas Koenig --- Jerry, I think you know more about this than I do, so please go ahead.

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-10-07 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #20 from Jerry DeLisle --- (In reply to Thomas Koenig from comment #19) > Created attachment 42320 [details] > patch which failes with dtio_14 and dtio_15, among others > > Well, this one doesn't work yet. Do you want to continue

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-10-07 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #19 from Thomas Koenig --- Created attachment 42320 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=42320=edit patch which failes with dtio_14 and dtio_15, among others Well, this one doesn't work yet.

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-10-07 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #18 from Dominique d'Humieres --- I do not see any timing difference between 7.2.1 (r253502) and trunk.

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-10-07 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #17 from Thomas Koenig --- I mean apart from the fact that this creates a huge memory leak :-)

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-10-07 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #16 from Thomas Koenig --- Jerry, what do you think of this approach? This creates a local copy of the gfc_unit, without putting it into the tree. Index: transfer.c

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-10-06 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #15 from Jerry DeLisle --- For my own baseline: gcc6: real 0m6.948s gcc7: real 0m9.906s gcc8: real 0m10.415s I backported removal of the caching mentioned in comment #14 to gcc7. The two should be identical except

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-10-06 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 --- Comment #14 from Jerry DeLisle --- The real issue is that to support DTIO with internal units I had to actually use a gfc_unit structure. Before DTIO we never did this. At the time of doing DTIO I did not have a 'better idea' since by the

[Bug libfortran/78549] [8 Regression] Very slow formatted internal file output

2017-10-06 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549 Thomas Koenig changed: What|Removed |Added Status|WAITING |NEW Target Milestone|---