]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: don't penalize ODOT and friends when inlining
authorIan Lance Taylor <iant@golang.org>
Sat, 19 Mar 2016 01:12:14 +0000 (18:12 -0700)
committerIan Lance Taylor <iant@golang.org>
Sun, 20 Mar 2016 17:25:02 +0000 (17:25 +0000)
commit060038bdd036e98622e22bc4538c7d192fb7217c
treed90be2a73f457653eb4d6f8f320619eed15e583b
parentd6d33f678dfe17e58b1d2f4b4b9d1487cd625c12
cmd/compile: don't penalize ODOT and friends when inlining

Historically ODOT and friends have been considered to cost an extra
budget point when deciding whether they should be inlined, because they
had an ONAME node that represented the name to the right of the dot.
This doesn't really make sense, as in general that symbol does not add
any extra instructions; it just affects the offset of the load or store
instruction.  And the ONAME node is gone now.  So, remove the extra
cost.

This does not pass toolstash -cmp, as it changes inlining decisions.
For example, mspan.init in runtime/mheap.go is now considered to be an
inlining candidate.

Change-Id: I5ad27f08c66fd5daa4c8472dd0795df989183f5e
Reviewed-on: https://go-review.googlesource.com/20891
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
src/cmd/compile/internal/gc/inl.go