diff options
| author | Douglas Wilson <douglas.wilson@gmail.com> | 2017-10-25 14:20:06 -0400 |
|---|---|---|
| committer | Ben Gamari <ben@smart-cactus.org> | 2017-10-25 15:47:25 -0400 |
| commit | 1c15d8ed112bccf2635d571767733b2a26d8fb21 (patch) | |
| tree | a952104e64fd898c8698a0c6e75c81c944a96651 /testsuite/tests/perf/compiler/all.T | |
| parent | 980e1270ed7f681ef666ca36fe291cfb8613348c (diff) | |
| download | haskell-1c15d8ed112bccf2635d571767733b2a26d8fb21.tar.gz | |
Fix space leak in BinIface.getSymbolTable
Replace a call to mapAccumR, which uses linear stack space, with a
gadget that uses constant space.
Remove an unused parameter from fromOnDiskName.
The tests T1292_imports and T4239 are now reporting imported names in a
different order. I don't completely understand why, but I presume it is
because the symbol tables are now read more strictly. The new order
seems better in T1792_imports, and equally random in T4239.
There are several performance test improvements.
Test Plan: ./validate
Reviewers: austin, bgamari
Reviewed By: bgamari
Subscribers: alexbiehl, rwbarton, thomie
Differential Revision: https://phabricator.haskell.org/D4124
Diffstat (limited to 'testsuite/tests/perf/compiler/all.T')
| -rw-r--r-- | testsuite/tests/perf/compiler/all.T | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/testsuite/tests/perf/compiler/all.T b/testsuite/tests/perf/compiler/all.T index b80900d0f0..41b2af8df9 100644 --- a/testsuite/tests/perf/compiler/all.T +++ b/testsuite/tests/perf/compiler/all.T @@ -658,7 +658,7 @@ test('T5837', # 2017-02-19 59161648 (x64/Windows) - Unknown # 2017-04-21 54985248 (x64/Windows) - Unknown - (wordsize(64), 56782344, 7)]) + (wordsize(64), 52089424, 7)]) # sample: 3926235424 (amd64/Linux, 15/2/2012) # 2012-10-02 81879216 # 2012-09-20 87254264 amd64/Linux @@ -695,6 +695,7 @@ test('T5837', # 2017-02-28 54151864 amd64/Linux Likely drift due to recent simplifier improvements # 2017-02-25 52625920 amd64/Linux Early inlining patch # 2017-09-06 56782344 amd64/Linux Drift manifest in unrelated LLVM patch + # 2017-10-24 52089424 amd64/linux Fix space leak in BinIface.getSymbolTable ], compile, ['-freduction-depth=50']) @@ -1114,10 +1115,11 @@ test('T12707', test('T12150', [ only_ways(['optasm']), compiler_stats_num_field('bytes allocated', - [(wordsize(64), 78300680, 5) + [(wordsize(64), 73769936, 5) # initial: 70773000 # 2017-08-25: 74358208 Refactor the Mighty Simplifier # 2017-08-25: 78300680 Drift + # 2017-10-25: 73769936 amd64/linux Fix space leak in BinIface.getSymbolTable ]), ], compile, |