FAQ
Reviewers: golang-dev_googlegroups.com, rsc, lvd2,

Message:
Hello golang-dev@googlegroups.com, rsc@golang.org, lvd@gmail.com (cc:
golang-dev@googlegroups.com),

I'd like you to review this change to
https://go.googlecode.com/hg/


Description:
cmd/gc: fix inlining internal error with T.Method calls.

The compiler was confused when inlining a T.Method(f()) call
where f returns multiple values: support for this was marked
as TODO.

Variadic calls are not supported but are not inlined either.
Add a test preventively for that case.

Fixes issue 4167.

Please review this at https://codereview.appspot.com/6871043/

Affected files:
M src/cmd/gc/inl.c
A test/fixedbugs/issue4167.go

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupgolang-dev @
categoriesgo
postedDec 1, '12 at 10:15p
activeDec 1, '12 at 10:15p
posts1
users1
websitegolang.org

1 user in discussion

Remyoudompheng: 1 post

People

Translate

site design / logo © 2022 Grokbase