code review 6810090: cmd/6g, cmd/8g: mark used registers in indirect addressing.

cmd/6g, cmd/8g: mark used registers in indirect addressing.

Fixes issue 4094.
Fixes issue 4353.

[remyoudompheng, 2012-11-06 23:19:28]

Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),

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

[remyoudompheng, 2012-11-06 23:22:58]

The fixes issue 4094 is a bit hypothetical, it is a bit hard to actually
reproduce. But in some sense, the issue is just as much hypothetical.

[dave_cheney.net, 2012-11-06 23:26:14]

Is it possible to include the sample code from 4353 as in src/test ?

On Wed, Nov 7, 2012 at 10:22 AM,  <remyoudompheng@gmail.com> wrote:
> The fixes issue 4094 is a bit hypothetical, it is a bit hard to actually
> reproduce. But in some sense, the issue is just as much hypothetical.
>
> http://codereview.appspot.com/6810090/

[remyoudompheng, 2012-11-06 23:37:11]

Hello golang-dev@googlegroups.com, dave@cheney.net (cc:
golang-dev@googlegroups.com),

Please take another look.

[dave_cheney.net, 2012-11-06 23:56:08]

Thank you for adding the test. I wanted to make sure that the test passed on
arm, and it does.

[rsc, 2012-11-07 14:22:34]

LGTM

Thanks very much for tracking that down.

[remyoudompheng, 2012-11-07 20:36:52]

*** Submitted as http://code.google.com/p/go/source/detail?r=aae11629f3e8 ***

cmd/6g, cmd/8g: mark used registers in indirect addressing.

Fixes issue 4094.
Fixes issue 4353.

R=golang-dev, dave, rsc
CC=golang-dev
http://codereview.appspot.com/6810090