qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-24 00:00:23 -07:00
Code Releases Activity

cmd/compile/internal/gc: remove dead code from stringtoarraylit

Browse Source 
The code path for []byte is unused.
Rename function to stringtoruneslit to reflect change in the behavior.

Note that removed code had a bug in it,
it used [0] index instead of [i] inside a loop body.

Change-Id: I58ece5d9d3835887b014446f8a7d3e7fc2fdcaa3
Reviewed-on: https://go-review.googlesource.com/c/125796
Run-TryBot: Iskander Sharipov <iskander.sharipov@intel.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
This commit is contained in:
Iskander Sharipov 2018-07-25 19:06:01 +03:00
parent 05166bf54d
commit 5ddb209120
1 changed files with 10 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/cmd/compile/internal/gc/typecheck.go
View File

@ -1719,14 +1719,14 @@ func typecheck1(n *Node, top int) *Node {
}
}
// do not use stringtoarraylit.
// do not convert to []byte literal. See CL 125796.
// generated code and compiler memory footprint is better without it.
case OSTRARRAYBYTE:
break
case OSTRARRAYRUNE:
if n.Left.Op == OLITERAL {
n = stringtoarraylit(n)
n = stringtoruneslit(n)
}
}
@ -3509,28 +3509,20 @@ func typecheckfunc(n *Node) {
}
}
// The result of stringtoarraylit MUST be assigned back to n, e.g.
// n.Left = stringtoarraylit(n.Left)
func stringtoarraylit(n *Node) *Node {
// The result of stringtoruneslit MUST be assigned back to n, e.g.
// n.Left = stringtoruneslit(n.Left)
func stringtoruneslit(n *Node) *Node {
if n.Left.Op != OLITERAL || n.Left.Val().Ctype() != CTSTR {
Fatalf("stringtoarraylit %v", n)
}
s := n.Left.Val().U.(string)
var l []*Node
if n.Type.Elem().Etype == TUINT8 {
// []byte
for i := 0; i < len(s); i++ {
l = append(l, nod(OKEY, nodintconst(int64(i)), nodintconst(int64(s[0]))))
}
} else {
// []rune
s := n.Left.Val().U.(string)
i := 0
for _, r := range s {
l = append(l, nod(OKEY, nodintconst(int64(i)), nodintconst(int64(r))))
i++
}
}
nn := nod(OCOMPLIT, nil, typenod(n.Type))
nn.List.Set(l)