]> Cypherpunks repositories - gostls13.git/commit
math: remove asm version of sincos everywhere, except 386
authorIlya Tocar <ilya.tocar@intel.com>
Thu, 20 Apr 2017 19:07:57 +0000 (14:07 -0500)
committerIlya Tocar <ilya.tocar@intel.com>
Mon, 24 Apr 2017 15:09:18 +0000 (15:09 +0000)
commitbc6459ac6ce1133cd0c59864d220d60f8fca40ef
tree615a45f6bd24820fa668b80803a4bbd809258f02
parent1737aef270a8954f3f8718beb55d6ddfc1cf5d21
math: remove asm version of sincos everywhere, except 386

We have dedicated asm implementation of sincos only on 386 and amd64,
on everything else we are just jumping to generic version.
However amd64 version is actually slower than generic one:

Sincos-6               34.4ns ± 0%   24.8ns ± 0%  -27.79%  (p=0.000 n=8+10)

So remove all sincos*.s and keep only generic and 386.

Updates #19819

Change-Id: I7eefab35743729578264f52f6d23ee2c227c92a5
Reviewed-on: https://go-review.googlesource.com/41200
Run-TryBot: Ilya Tocar <ilya.tocar@intel.com>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
src/math/sincos.go
src/math/sincos_386.go [new file with mode: 0644]
src/math/sincos_amd64.s [deleted file]
src/math/sincos_amd64p32.s [deleted file]
src/math/sincos_arm.s [deleted file]
src/math/stubs_arm64.s
src/math/stubs_mips64x.s
src/math/stubs_mipsx.s
src/math/stubs_ppc64x.s
src/math/stubs_s390x.s