[commit: ghc] master: Refactor floating of bindings (fiBind) (e4188b5)

[git at git.haskell.org]

Repository : ssh://git@git.haskell.org/ghc

On branch  : master
Link       : http://ghc.haskell.org/trac/ghc/changeset/e4188b538bfc879b201d416cf1d68ff7072c577f/ghc

>---------------------------------------------------------------

commit e4188b538bfc879b201d416cf1d68ff7072c577f
Author: Simon Peyton Jones <simonpj at microsoft.com>
Date:   Sun Feb 26 21:45:00 2017 -0500

    Refactor floating of bindings (fiBind)
    
    This is just a local refactoring.
    
    I originally planned to try floating top-level bindings inwards,
    but I backed off from that leaving only this (harmless) refactoring,
    which has no behavioural effect.
    
    I also make FloatIn into a ModGuts -> ModGuts function; again not
    necessary now, but no harm either.
    
    My attempt also used the new function CoreFVs.freeVarsBind; but
    that too is a plausible refactorig of freeVars, so I left it in too.
    
    Reviewers: austin, bgamari
    
    Subscribers: thomie
    
    Differential Revision: https://phabricator.haskell.org/D3180


>---------------------------------------------------------------

e4188b538bfc879b201d416cf1d68ff7072c577f
 compiler/coreSyn/CoreFVs.hs            |  57 +++++------
 compiler/simplCore/FloatIn.hs          | 173 ++++++++++++++++++---------------
 compiler/simplCore/SimplCore.hs        |   2 +-
 testsuite/tests/perf/space_leaks/all.T |   6 +-
 4 files changed, 129 insertions(+), 109 deletions(-)

Diff suppressed because of size. To see it, use:

    git diff-tree --root --patch-with-stat --no-color --find-copies-harder --ignore-space-at-eol --cc e4188b538bfc879b201d416cf1d68ff7072c577f