• Josh Bleecher Snyder's avatar
    cmd/compile: speed up hot phi insertion code · 61fb2f6d
    Josh Bleecher Snyder authored
    This speeds up compilation of the code in #8225 by 25%-30%.
    The complexity of the algorithm is unchanged,
    but this shrinks the constant factor so much that it doesn't matter,
    even the size of the giant type switch gets scaled up dramatically.
    
    name       old time/op      new time/op      delta
    Template        218ms ± 5%       217ms ±10%    ~           (p=0.163 n=27+30)
    Unicode        98.2ms ± 6%      97.7ms ±10%    ~           (p=0.150 n=27+29)
    GoTypes         654ms ± 5%       650ms ± 5%    ~           (p=0.350 n=30+30)
    Compiler        2.70s ± 4%       2.68s ± 3%    ~           (p=0.128 n=30+29)
    
    name       old user-ns/op   new user-ns/op   delta
    Template   276user-ms ± 6%  271user-ms ± 7%  -1.83%        (p=0.003 n=29+28)
    Unicode    138user-ms ± 5%  137user-ms ± 4%    ~           (p=0.071 n=27+27)
    GoTypes    881user-ms ± 4%  877user-ms ± 4%    ~           (p=0.423 n=30+30)
    Compiler   3.76user-s ± 4%  3.72user-s ± 2%  -0.84%        (p=0.028 n=30+29)
    
    name       old alloc/op     new alloc/op     delta
    Template       40.7MB ± 0%      40.7MB ± 0%    ~           (p=0.936 n=30+30)
    Unicode        30.8MB ± 0%      30.8MB ± 0%    ~           (p=0.859 n=28+30)
    GoTypes         123MB ± 0%       123MB ± 0%    ~           (p=0.273 n=30+30)
    Compiler        472MB ± 0%       472MB ± 0%    ~           (p=0.432 n=30+30)
    
    name       old allocs/op    new allocs/op    delta
    Template         401k ± 1%        401k ± 1%    ~           (p=0.859 n=30+30)
    Unicode          331k ± 0%        331k ± 1%    ~           (p=0.823 n=28+30)
    GoTypes         1.24M ± 0%       1.24M ± 0%    ~           (p=0.286 n=30+30)
    Compiler        4.26M ± 0%       4.26M ± 0%    ~           (p=0.359 n=30+30)
    
    Change-Id: Ia850065a9a84c07a5b0b4e23c1758b5679498da7
    Reviewed-on: https://go-review.googlesource.com/36112
    Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
    TryBot-Result: Gobot Gobot <gobot@golang.org>
    Reviewed-by: 's avatarKeith Randall <khr@golang.org>
    61fb2f6d
phi.go 14.5 KB