diff options
| author | Matthew Dempsky <mdempsky@google.com> | 2022-06-23 13:07:32 -0700 |
|---|---|---|
| committer | Matthew Dempsky <mdempsky@google.com> | 2022-06-23 23:03:20 +0000 |
| commit | 09a838ad86880150f4e297f7b2dec7c7d116623b (patch) | |
| tree | 629a726054d00f35a215899702b3c23a06448900 /src/cmd/compile/internal/noder | |
| parent | 421e9e9db21363a196f9c5d736749a6754803bff (diff) | |
| download | go-09a838ad86880150f4e297f7b2dec7c7d116623b.tar.xz | |
[dev.unified] cmd/compile: rename haveRType and implicitExpr
This CL renames:
1. "haveRType" to "hasRType", suggested by drchase@ during review of
CL 413358; and
2. "implicitExpr" to "implicitConvExpr", suggested by khr@ during
review of CL 413396.
Change-Id: Ibb4deae20908d960706640991ea44d1b9c0b9e3c
Reviewed-on: https://go-review.googlesource.com/c/go/+/413854
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
Diffstat (limited to 'src/cmd/compile/internal/noder')
| -rw-r--r-- | src/cmd/compile/internal/noder/writer.go | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/cmd/compile/internal/noder/writer.go b/src/cmd/compile/internal/noder/writer.go index ec744f4122..0537d1d3b2 100644 --- a/src/cmd/compile/internal/noder/writer.go +++ b/src/cmd/compile/internal/noder/writer.go @@ -1088,7 +1088,7 @@ func (w *writer) stmt1(stmt syntax.Stmt) { if stmt.Op != syntax.Shl && stmt.Op != syntax.Shr { typ = w.p.typeOf(stmt.Lhs) } - w.implicitExpr(stmt, typ, stmt.Rhs) + w.implicitConvExpr(stmt, typ, stmt.Rhs) default: w.assignStmt(stmt, stmt.Lhs, stmt.Rhs) @@ -1146,7 +1146,7 @@ func (w *writer) stmt1(stmt syntax.Stmt) { resultTypes := w.sig.Results() if len(exprs) == resultTypes.Len() { for i, expr := range exprs { - w.implicitExpr(stmt, resultTypes.At(i).Type(), expr) + w.implicitConvExpr(stmt, resultTypes.At(i).Type(), expr) } } else if len(exprs) == 0 { // ok: bare "return" with named result parameters @@ -1166,7 +1166,7 @@ func (w *writer) stmt1(stmt syntax.Stmt) { w.Code(stmtSend) w.pos(stmt) w.expr(stmt.Chan) - w.implicitExpr(stmt, chanType.Elem(), stmt.Value) + w.implicitConvExpr(stmt, chanType.Elem(), stmt.Value) case *syntax.SwitchStmt: w.Code(stmtSwitch) @@ -1263,7 +1263,7 @@ func (w *writer) assignStmt(pos poser, lhs0, rhs0 syntax.Expr) { dstType = w.p.typeOf(dst) } - w.implicitExpr(pos, dstType, expr) + w.implicitConvExpr(pos, dstType, expr) } } else if len(rhs) == 0 { // ok: variable declaration without values @@ -1498,7 +1498,7 @@ func (w *writer) expr(expr syntax.Expr) { w.Code(exprIndex) w.expr(expr.X) w.pos(expr) - w.implicitExpr(expr, keyType, expr.Index) + w.implicitConvExpr(expr, keyType, expr.Index) case *syntax.SliceExpr: w.Code(exprSlice) @@ -1607,7 +1607,7 @@ func (w *writer) expr(expr syntax.Expr) { } else { paramType = paramTypes.At(i).Type() } - w.implicitExpr(expr, paramType, arg) + w.implicitConvExpr(expr, paramType, arg) } w.Bool(expr.HasDots) @@ -1621,10 +1621,10 @@ func (w *writer) optExpr(expr syntax.Expr) { } } -// implicitExpr is like expr, but if dst is non-nil and different from +// implicitConvExpr is like expr, but if dst is non-nil and different from // expr's type, then an implicit conversion operation is inserted at // pos. -func (w *writer) implicitExpr(pos poser, dst types2.Type, expr syntax.Expr) { +func (w *writer) implicitConvExpr(pos poser, dst types2.Type, expr syntax.Expr) { src := w.p.typeOf(expr) if dst != nil && !types2.Identical(src, dst) { if !types2.AssignableTo(src, dst) { @@ -1682,12 +1682,12 @@ func (w *writer) compLit(lit *syntax.CompositeLit) { if kv, ok := elem.(*syntax.KeyValueExpr); w.Bool(ok) { // use position of expr.Key rather than of elem (which has position of ':') w.pos(kv.Key) - w.implicitExpr(kv.Key, keyType, kv.Key) + w.implicitConvExpr(kv.Key, keyType, kv.Key) elem = kv.Value } } w.pos(elem) - w.implicitExpr(elem, elemType, elem) + w.implicitConvExpr(elem, elemType, elem) } } |
