• David Chase's avatar
    cmd/compile: mark temps with new AutoTemp flag, and use it. · 9c066bab
    David Chase authored
    This is an extension of
    https://go-review.googlesource.com/c/31662/
    to mark all the temporaries, not just the ssa-generated ones.
    
    Before-and-after ls -l `go tool -n compile` shows a 3%
    reduction in size (or rather, a prior 3% inflation for
    failing to filter temps out properly.)
    
    Replaced name-dependent "is it a temp?" tests with calls to
    *Node.IsAutoTmp(), which depends on AutoTemp.  Also replace
    calls to istemp(n) with n.IsAutoTmp(), to reduce duplication
    and clean up function name space.  Generated temporaries
    now come with a "." prefix to avoid (apparently harmless)
    clashes with legal Go variable names.
    
    Fixes #17644.
    Fixes #17240.
    
    Change-Id: If1417f29c79a7275d7303ddf859b51472890fd43
    Reviewed-on: https://go-review.googlesource.com/32255
    Run-TryBot: David Chase <drchase@google.com>
    TryBot-Result: Gobot Gobot <gobot@golang.org>
    Reviewed-by: 's avatarMatthew Dempsky <mdempsky@google.com>
    9c066bab
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...
favicon.ico Loading commit data...
robots.txt Loading commit data...