• Russ Cox's avatar
    cmd/internal/gc: move cgen, regalloc, et al to portable code · b115c35e
    Russ Cox authored
    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/7853Reviewed-by: 's avatarBrad Fitzpatrick <bradfitz@golang.org>
    b115c35e
galign.go 2.19 KB