mirror of
https://github.com/golang/go
synced 2024-09-06 00:19:45 +00:00
![Matthew Dempsky](/assets/img/avatar_default.png)
This is a re-attempt at CL 153841, which caused two regressions: 1. crypto/ecdsa failed to build with -gcflags=-l=4. This was because when "t1, t2, ... := g(); f(t1, t2, ...)" was exported, we were losing the first assignment from the call's Ninit field. 2. net/http/pprof failed to run with -gcflags=-N. This is due to a conflict with CL 159717: as of that CL, package-scope initialization statements are executed within the "init.ializer" function, rather than the "init" function, and the generated temp variables need to be moved accordingly too. [Rest of description is as before.] This CL moves order.go's copyRet logic for rewriting f(g()) into t1, t2, ... := g(); f(t1, t2, ...) earlier into typecheck. This allows the rest of the compiler to stop worrying about multi-value functions appearing outside of OAS2FUNC nodes. This changes compiler behavior in a few observable ways: 1. Typechecking error messages for builtin functions now use general case error messages rather than unnecessarily differing ones. 2. Because f(g()) is rewritten before inlining, saved inline bodies now see the rewritten form too. This could be addressed, but doesn't seem worthwhile. 3. Most notably, this simplifies escape analysis and fixes a memory corruption issue in esc.go. See #29197 for details. Fixes #15992. Fixes #29197. Change-Id: I930b10f7e27af68a0944d6c9bfc8707c3fab27a4 Reviewed-on: https://go-review.googlesource.com/c/go/+/166983 Run-TryBot: Matthew Dempsky <mdempsky@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Robert Griesemer <gri@golang.org>
69 lines
1.4 KiB
Go
69 lines
1.4 KiB
Go
// errorcheck
|
|
|
|
// Copyright 2010 The Go Authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style
|
|
// license that can be found in the LICENSE file.
|
|
|
|
// Verify that incorrect invocations of the complex predeclared function are detected.
|
|
// Does not compile.
|
|
|
|
package main
|
|
|
|
type (
|
|
Float32 float32
|
|
Float64 float64
|
|
Complex64 complex64
|
|
Complex128 complex128
|
|
)
|
|
|
|
var (
|
|
f32 float32
|
|
f64 float64
|
|
F32 Float32
|
|
F64 Float64
|
|
|
|
c64 complex64
|
|
c128 complex128
|
|
C64 Complex64
|
|
C128 Complex128
|
|
)
|
|
|
|
func F1() int {
|
|
return 1
|
|
}
|
|
|
|
func F3() (int, int, int) {
|
|
return 1, 2, 3
|
|
}
|
|
|
|
func main() {
|
|
// ok
|
|
c64 = complex(f32, f32)
|
|
c128 = complex(f64, f64)
|
|
|
|
_ = complex128(0) // ok
|
|
_ = complex(f32, f64) // ERROR "complex"
|
|
_ = complex(f64, f32) // ERROR "complex"
|
|
_ = complex(f32, F32) // ERROR "complex"
|
|
_ = complex(F32, f32) // ERROR "complex"
|
|
_ = complex(f64, F64) // ERROR "complex"
|
|
_ = complex(F64, f64) // ERROR "complex"
|
|
|
|
_ = complex(F1()) // ERROR "not enough arguments"
|
|
_ = complex(F3()) // ERROR "too many arguments"
|
|
|
|
_ = complex() // ERROR "not enough arguments"
|
|
|
|
c128 = complex(f32, f32) // ERROR "cannot use"
|
|
c64 = complex(f64, f64) // ERROR "cannot use"
|
|
|
|
c64 = complex(1.0, 2.0) // ok, constant is untyped
|
|
c128 = complex(1.0, 2.0)
|
|
C64 = complex(1.0, 2.0)
|
|
C128 = complex(1.0, 2.0)
|
|
|
|
C64 = complex(f32, f32) // ERROR "cannot use"
|
|
C128 = complex(f64, f64) // ERROR "cannot use"
|
|
|
|
}
|