diff options
author | Michal Terepeta <michal.terepeta@gmail.com> | 2016-11-29 13:31:28 -0500 |
---|---|---|
committer | Ben Gamari <ben@smart-cactus.org> | 2016-11-29 14:39:55 -0500 |
commit | da5a61eb15237d406e4d1fb4390d47602ee4c34c (patch) | |
tree | 9eae546b42ac12561da9fc67f066814977dc55ac /compiler/codeGen | |
parent | 52222f9bf705ad64bc4a212088d153d8918b6173 (diff) | |
download | haskell-da5a61eb15237d406e4d1fb4390d47602ee4c34c.tar.gz |
Minor cleanup of foldRegs{Used,Defd}
This makes the two functions strict in the accumulator - it seems that
there are only two users of those functions: `CmmLive` and `CmmSink`
and in both cases the strict fold fits better.
The commit also removes a few unused functions (`filterRegsUsed`),
instances (for `Maybe` and `RegSet`) and gets rid of unnecessary
inculde of `HsVersions.h`.
The performance effect of avoiding unnecessary thunks is mostly
negligible, although we do allocate a tiny bit less (nofib's section
on compile allocations):
```
-1 s.d. ----- -0.2%
+1 s.d. ----- -0.1%
Average ----- -0.2%
```
Signed-off-by: Michal Terepeta <michal.terepeta@gmail.com>
Test Plan: validate
Reviewers: simonmar, austin, bgamari
Reviewed By: bgamari
Subscribers: thomie
Differential Revision: https://phabricator.haskell.org/D2723
Diffstat (limited to 'compiler/codeGen')
0 files changed, 0 insertions, 0 deletions