]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: remove statement marks from secondary calls
authorDavid Chase <drchase@google.com>
Mon, 29 Jul 2019 20:23:31 +0000 (16:23 -0400)
committerDavid Chase <drchase@google.com>
Fri, 4 Oct 2019 20:41:52 +0000 (20:41 +0000)
commitc450ace12c657e3953d79975c04f51605395cd50
tree76faeebeefa1d8b68f66f737cedee8438fd33f5d
parent9a926911fea73017a25d6d38035946c59cf1b047
cmd/compile: remove statement marks from secondary calls

Calls are code-generated in an alternate path that inherits
its positions from values, not from *SSAGenState.  The
default position on *SSAGenState was marked as not-a-statement,
but this was not applied to the value itself, leading to
spurious "is statement" marks in the output (convention:
after code generation in the compiler, everything is either
definitely a statement or definitely not a statement, nothing
is in the undetermined state).

This CL causes a 35 statement regression in ssa/stmtlines_test.
This is down from the earlier 150 because of all the other
CLs preceding this one that deal with the root causes of the
missing lines (repeated lines on nested calls hid missing lines).

This also removes some line repeats from ssa/debug_test.

Change-Id: Ie9a507bd5447e906b35bbd098e3295211df2ae01
Reviewed-on: https://go-review.googlesource.com/c/go/+/188018
Run-TryBot: David Chase <drchase@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Jeremy Faller <jeremy@golang.org>
src/cmd/compile/internal/gc/ssa.go
src/cmd/compile/internal/ssa/testdata/hist.dlv-opt.nexts
src/cmd/compile/internal/ssa/testdata/hist.gdb-dbg.nexts
src/cmd/compile/internal/ssa/testdata/hist.gdb-opt.nexts