Stowage/go
2
0
Fork 0
mirror of https://github.com/golang/go.git synced 2025-12-08 06:10:04 +00:00
Code Issues Projects Releases Packages Wiki Activity

cmd/compile: reenable name preservation on copies in expand_calls

Browse source 
This reverts CL 311829, and reenables CL 309330. The issue
should be fixed in the previous CL.

Change-Id: I69db0565c72470a1814f135d8f8ec62c781bfc5a
Reviewed-on: https://go-review.googlesource.com/c/go/+/312094
Trust: Cherry Zhang <cherryyz@google.com>
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: David Chase <drchase@google.com>
Reviewed-by: Than McIntosh <thanm@google.com>
This commit is contained in:
Cherry Zhang 2021-04-20 21:07:23 -04:00
parent daee726a0b
commit e5bc4f2a77
2 changed files with 9 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/cmd/compile/internal/ssa/expand_calls.go
View file

@ -1452,7 +1452,6 @@ func expandCalls(f *Func) {
// Step 6: elide any copies introduced. // Step 6: elide any copies introduced.
// Update named values. // Update named values.
if false { // TODO: reeanable. It caused compiler OOMing on large input.
for _, name := range f.Names { for _, name := range f.Names {
values := f.NamedValues[name] values := f.NamedValues[name]
for i, v := range values { for i, v := range values {
@ -1465,7 +1464,6 @@ func expandCalls(f *Func) {
} }
} }
} }
}
for _, b := range f.Blocks { for _, b := range f.Blocks {
for _, v := range b.Values { for _, v := range b.Values {
for i, a := range v.Args { for i, a := range v.Args {

5
test/live_regabi.go
View file

@ -1,8 +1,5 @@
// errorcheckwithauto -0 -l -live -wb=0 -d=ssa/insert_resched_checks/off // errorcheckwithauto -0 -l -live -wb=0 -d=ssa/insert_resched_checks/off
// +build amd64,goexperiment.regabidefer,goexperiment.regabiargs,ignore // +build amd64,goexperiment.regabidefer,goexperiment.regabiargs
// Disabled for now. The compiler sometimes has bad name-value association
// for args, causing args appears as autotmps.
// Copyright 2014 The Go Authors. All rights reserved. // Copyright 2014 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style // Use of this source code is governed by a BSD-style