]> Cypherpunks repositories - gostls13.git/commit
[dev.typeparams] cmd/compile: inlining tweaks for toolstash
authorMatthew Dempsky <mdempsky@google.com>
Mon, 24 May 2021 11:41:58 +0000 (04:41 -0700)
committerMatthew Dempsky <mdempsky@google.com>
Wed, 26 May 2021 23:50:54 +0000 (23:50 +0000)
commit1ec056244e1a058ea3a21f0abd1165d710398416
tree3ce7c908aa3044f6f86f3c2841f6f31fa5e578c2
parent6da1661371410c46af84c578d644052894226314
[dev.typeparams] cmd/compile: inlining tweaks for toolstash

This CL makes to minor changes motivated by making it easier to make
large-scale changes to the inliner while satisfying toolstash -cmp:

1. When creating inlining variables, make sure to preserve the
AutoTemp flag. This is necessary so that temporary variables
introduced by rewriting f(g()) calls during typecheck stay autotemp
after inlining and are (correctly) omitted from DWARF debugging
information.

2. When sorting variables for stack frame layout, use a stable
sort. This ensures that layout is insensitive to whether deadcode
elimination happens before or after inlining.

Change-Id: I672e752a873c7e16749b9873fd6573607e074309
Reviewed-on: https://go-review.googlesource.com/c/go/+/323011
Trust: Matthew Dempsky <mdempsky@google.com>
Trust: Dan Scales <danscales@google.com>
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Dan Scales <danscales@google.com>
src/cmd/compile/internal/inline/inl.go
src/cmd/compile/internal/ssagen/pgen.go