]> Cypherpunks repositories - gostls13.git/commit
cmd/internal/gc: move cgen, regalloc, et al to portable code
authorRuss Cox <rsc@golang.org>
Wed, 18 Mar 2015 21:26:36 +0000 (17:26 -0400)
committerRuss Cox <rsc@golang.org>
Fri, 20 Mar 2015 20:03:52 +0000 (20:03 +0000)
commitb115c35ee337ac6026539802d2ff085949dd1919
tree49868c29d692bcf7d0e5a46b44bcbf17e354afa6
parent11dba2ec2d8fcedc1da0103925c254586ef51120
cmd/internal/gc: move cgen, regalloc, et al to portable code

This CL moves the bulk of the code that has been copy-and-pasted
since the initial 386 port back into a shared place, cutting 5 copies to 1.

The motivation here is not cleanup per se but instead to reduce the
cost of introducing changes in shared concepts like regalloc or general
expression evaluation. For example, a change after this one will
implement x.(*T) without a call into the runtime. This CL makes that
followup work 5x easier.

The single copy still has more special cases for architecture details
than I'd like, but having them called out explicitly like this at least
opens the door to generalizing the conditions and smoothing out
the distinctions in the future.

This is a LARGE CL. I started by trying to pull in one function at a time
in a sequence of CLs and it became clear that everything was so
interrelated that it had to be moved as a whole. Apologies for the size.

It is not clear how many more releases this code will matter for;
eventually it will be replaced by Keith's SSA work. But as noted above,
the deduplication was necessary to reduce the cost of working on
the current code while we have it.

Passes tests on amd64, 386, arm, and ppc64le.
Can build arm64 binaries but not tested there.
Being able to build binaries means it is probably very close.

Change-Id: I735977f04c0614f80215fb12966dfe9bbd1f5861
Reviewed-on: https://go-review.googlesource.com/7853
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
39 files changed:
src/cmd/5g/cgen.go
src/cmd/5g/cgen64.go
src/cmd/5g/galign.go
src/cmd/5g/gg.go [deleted file]
src/cmd/5g/ggen.go
src/cmd/5g/gsubr.go
src/cmd/5g/peep.go
src/cmd/6g/cgen.go
src/cmd/6g/galign.go
src/cmd/6g/ggen.go
src/cmd/6g/gsubr.go
src/cmd/7g/cgen.go
src/cmd/7g/galign.go
src/cmd/7g/ggen.go
src/cmd/7g/gsubr.go
src/cmd/8g/cgen.go
src/cmd/8g/cgen64.go
src/cmd/8g/galign.go
src/cmd/8g/ggen.go
src/cmd/8g/gsubr.go
src/cmd/8g/peep.go
src/cmd/9g/cgen.go
src/cmd/9g/galign.go
src/cmd/9g/gg.go [deleted file]
src/cmd/9g/ggen.go
src/cmd/9g/gsubr.go
src/cmd/internal/gc/cgen.go [new file with mode: 0644]
src/cmd/internal/gc/cplx.go
src/cmd/internal/gc/gen.go
src/cmd/internal/gc/go.go
src/cmd/internal/gc/gsubr.go
src/cmd/internal/gc/pgen.go
src/cmd/internal/gc/popt.go
src/cmd/internal/gc/reg.go
src/cmd/internal/gc/syntax.go
src/cmd/internal/obj/arm64/asm7.go
src/cmd/internal/obj/go.go
src/cmd/internal/obj/sym.go
src/cmd/internal/obj/x86/obj6.go