Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

[WIP] Optimize the gcSections pass #605

Merged
merged 24 commits into from
Apr 24, 2020
Merged

[WIP] Optimize the gcSections pass #605

merged 24 commits into from
Apr 24, 2020

Conversation

gkaracha
Copy link
Member

No description provided.

asterius/src/Asterius/Types.hs Outdated Show resolved Hide resolved
@gkaracha gkaracha force-pushed the wip-fix-issue-568 branch from daf9327 to 72a99b8 Compare April 23, 2020 07:57
@gkaracha gkaracha linked an issue Apr 23, 2020 that may be closed by this pull request
@gkaracha gkaracha closed this Apr 24, 2020
@gkaracha gkaracha reopened this Apr 24, 2020
@gkaracha
Copy link
Member Author

Preliminary results

@TerrorJack, preliminary comparison between HEAD and master:

diff master/ahc.prof HEAD/ahc.prof shows:

<       total time  =        0.24 secs   (245 ticks @ 1000 us, 1 processor)
<       total alloc = 130,608,824 bytes  (excludes profiling overheads)
---
>       total time  =        0.24 secs   (244 ticks @ 1000 us, 1 processor)
>       total alloc = 132,291,880 bytes  (excludes profiling overheads)

diff master/ahc-link.prof HEAD/ahc-link.prof shows

<       total time  =       48.00 secs   (47998 ticks @ 1000 us, 1 processor)
<       total alloc = 3,320,867,416 bytes  (excludes profiling overheads)
---
>       total time  =       45.61 secs   (45606 ticks @ 1000 us, 1 processor)
>       total alloc = 3,320,989,152 bytes  (excludes profiling overheads)

diff master/ahc-ld.prof HEAD/ahc-ld.prof shows:

<       total time  =       32.17 secs   (32166 ticks @ 1000 us, 1 processor)
<       total alloc = 125,277,473,792 bytes  (excludes profiling overheads)
---
>       total time  =        9.68 secs   (9675 ticks @ 1000 us, 1 processor)
>       total alloc = 111,164,588,648 bytes  (excludes profiling overheads)

@gkaracha gkaracha force-pushed the wip-fix-issue-568 branch from 52ff1b7 to 55ef8a7 Compare April 24, 2020 13:16
@gkaracha gkaracha marked this pull request as ready for review April 24, 2020 13:17
@gkaracha
Copy link
Member Author

@TerrorJack I think it's ready for review now 🙂

@gkaracha
Copy link
Member Author

master vs HEAD gives:
diff master/ahc.prof HEAD/ahc.prof:

<       total time  =        0.25 secs   (246 ticks @ 1000 us, 1 processor)
<       total alloc = 130,610,000 bytes  (excludes profiling overheads)
---
>       total time  =        0.28 secs   (283 ticks @ 1000 us, 1 processor)
>       total alloc = 132,159,648 bytes  (excludes profiling overheads)

diff master/ahc-link.prof HEAD/ahc-link.prof:

<       total time  =       45.99 secs   (45991 ticks @ 1000 us, 1 processor)
<       total alloc = 3,320,876,440 bytes  (excludes profiling overheads)
---
>       total time  =       47.77 secs   (47772 ticks @ 1000 us, 1 processor)
>       total alloc = 3,321,019,664 bytes  (excludes profiling overheads)

diff master/ahc-ld.prof HEAD/ahc-ld.prof :

<       total time  =       32.59 secs   (32585 ticks @ 1000 us, 1 processor)
<       total alloc = 125,276,180,272 bytes  (excludes profiling overheads)
---
>       total time  =       11.17 secs   (11166 ticks @ 1000 us, 1 processor)
>       total alloc = 111,162,737,872 bytes  (excludes profiling overheads)

I think this is good to go @TerrorJack! 🙂

@TerrorJack TerrorJack merged commit 5eaec8f into master Apr 24, 2020
@TerrorJack TerrorJack deleted the wip-fix-issue-568 branch April 24, 2020 16:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Optimize the gc-sections pass
2 participants