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
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
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
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
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
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549
Thomas Koenig changed:
What|Removed |Added
CC||manfred99 at gmx dot ch
--- Comment #22
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.
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
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.
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.
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 :-)
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
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
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
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78549
Thomas Koenig changed:
What|Removed |Added
Status|WAITING |NEW
Target Milestone|---
15 matches
Mail list logo