Important performance wibble to callSiteInline (the n_vals_wanted > 0 thing)
authorsimonpj@microsoft.com <unknown>
Tue, 9 Sep 2008 15:50:11 +0000 (15:50 +0000)
committersimonpj@microsoft.com <unknown>
Tue, 9 Sep 2008 15:50:11 +0000 (15:50 +0000)
commite71d6d1f458685b6a20f6d02433667be1d4f7a26
tree4089011fbef5450292a54f2f18c9d73edbce5101
parent43a120b706a5eece6624ca4907af89fc9a480c5e
Important performance wibble to callSiteInline (the n_vals_wanted > 0 thing)

See Note [Inlining in ArgCtxt].  This very small change gives quite a
big performance win. Just showing the bigger ones:

        Program           Size    Allocs   Runtime
--------------------------------------------------------------------------------
           anna          -0.7%     -4.3%      0.15
       cichelli          -0.6%     -6.4%      0.15
         fulsom          -0.4%    -18.5%     -8.1%
            gcd          -0.6%    -12.0%      0.06
        integer          -0.6%    -16.2%     -8.4%
          power          -0.7%    -19.3%     -4.8%
--------------------------------------------------------------------------------
            Min          -0.7%    -19.3%    -15.7%
            Max          -0.1%     +0.1%     +5.7%
 Geometric Mean          -0.6%     -1.9%     -4.3%

The original change was to improve a case that Roman found (see test
eyeball/inline1) but that seems to work ok now anyway.
compiler/coreSyn/CoreUnfold.lhs