• Keith Randall's avatar
    [release-branch.go1.8] cmd/compile: zero ambiguously live variables at VARKILLs · e396667b
    Keith Randall authored
    This is a redo of CL 41076 backported to the 1.8 release branch.
    There were major conflicts, so I had to basically rewrite it again
    from scratch.  The way Progs are allocated changed.  Liveness analysis
    and Prog generation got reordered.  Liveness analysis changed from
    running on gc.BasicBlock to ssa.Block.  All that makes the logic quite
    a bit different.
    
    Please review carefully.
    
    From CL 41076:
    
    At VARKILLs, zero a variable if it is ambiguously live.
    After the VARKILL anything this variable references
    might be collected. If it were to become live again later,
    the GC will see references to already-collected objects.
    
    We don't know a variable is ambiguously live until very
    late in compilation (after lowering, register allocation, ...),
    so it is hard to generate the code in an arch-independent way.
    We also have to be careful not to clobber any registers.
    Fortunately, this almost never happens so performance is ~irrelevant.
    
    There are only 2 instances where this triggers in the stdlib.
    
    Fixes #20029
    
    Change-Id: Ibb757eec58ee07f40df5e561b19d315684dc4bda
    Reviewed-on: https://go-review.googlesource.com/43998
    Run-TryBot: Keith Randall <khr@golang.org>
    TryBot-Result: Gobot Gobot <gobot@golang.org>
    Reviewed-by: 's avatarMatthew Dempsky <mdempsky@google.com>
    e396667b
Name
Last commit
Last update
.github Loading commit data...
api Loading commit data...
doc Loading commit data...
lib/time Loading commit data...
misc Loading commit data...
src Loading commit data...
test Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
AUTHORS Loading commit data...
CONTRIBUTING.md Loading commit data...
CONTRIBUTORS Loading commit data...
LICENSE Loading commit data...
PATENTS Loading commit data...
README.md Loading commit data...
VERSION Loading commit data...
favicon.ico Loading commit data...
robots.txt Loading commit data...