FAQ
Reviewers: rsc, golang-dev_googlegroups.com,

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

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


Description:
cmd/6g, cmd/8g: fix two "out of fixed registers" cases.

In two cases, registers were allocated too early resulting
in exhausting of available registers when nesting these
operations.

The case of method calls was due to missing cases in igen,
which only makes calls but doesn't allocate a register for
the result.

The case of 8-bit multiplication was due to a wrong order
in register allocation when Ullman numbers were bigger on the
RHS.

Fixes issue 3907.
Fixes issue 4156.

Please review this at http://codereview.appspot.com/6560054/

Affected files:
M src/cmd/6g/cgen.c
M src/cmd/6g/ggen.c
M src/cmd/8g/cgen.c
M src/cmd/8g/ggen.c

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupgolang-dev @
categoriesgo
postedSep 26, '12 at 6:38p
activeSep 26, '12 at 7:17p
posts3
users2
websitegolang.org

2 users in discussion

Remyoudompheng: 2 posts Rsc: 1 post

People

Translate

site design / logo © 2022 Grokbase