runtime: arm: abort if VFPv3 support missing
Fixes issue 3456.
This proposal is a reformulation of CL 5987063. This CL resets
the default GOARM value to 6 and allows the use of the VFPv3
optimisation if GOARM=7. Binaries built with this CL in place
will abort if GOARM=7 was used and the target host does not
support VFPv3.
Hello minux.ma@gmail.com, rsc@golang.org, ajstarks@gmail.com (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go
LGTM with some minor comment suggestions. sorry for the late review. http://codereview.appspot.com/6501099/diff/7/src/cmd/5l/asm.c File src/cmd/5l/asm.c (right): ...
Thank you very much for your comments. PTAL. http://codereview.appspot.com/6501099/diff/7/src/cmd/5l/asm.c File src/cmd/5l/asm.c (right): http://codereview.appspot.com/6501099/diff/7/src/cmd/5l/asm.c#newcode2228 src/cmd/5l/asm.c:2228: // ...
*** Submitted as http://code.google.com/p/go/source/detail?r=bfad57cc7171 ***
runtime: arm: abort if VFPv3 support missing
Fixes issue 3456.
This proposal is a reformulation of CL 5987063. This CL resets
the default GOARM value to 6 and allows the use of the VFPv3
optimisation if GOARM=7. Binaries built with this CL in place
will abort if GOARM=7 was used and the target host does not
support VFPv3.
R=minux.ma, rsc, ajstarks
CC=golang-dev
http://codereview.appspot.com/6501099
Issue 6501099: code review 6501099: runtime: arm: abort if VFPv3 support missing
(Closed)
Created 13 years ago by dave_cheney.net
Modified 13 years ago
Reviewers:
Base URL:
Comments: 8