Age | Commit message (Collapse) | Author | Files | Lines |
|
Also improve the implementations of Equals and Compare.
R=rsc
CC=golang-dev
http://codereview.appspot.com/969047
|
|
Fixes issue 722.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1065041
|
|
When trying to regenerate src/pkg/runtime/darwin/386/defs.h
on a 64 bit capable Snow Leopard (OS X 10.6.3) system I
needed to add -f -m32 to godefs, as this OS and hardware
combination defaults to 64 bit compilation.
For safety, make the same change to the 32 bit FreeBSD
instructions in .../freebsd/defs.c. (Tested OK and no
problems introduced.)
R=golang-dev, rsc
CC=golang-dev
http://codereview.appspot.com/1052042
Committer: Russ Cox <rsc@golang.org>
|
|
TBR=r
CC=golang-dev
http://codereview.appspot.com/1063042
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/979043
Committer: Russ Cox <rsc@golang.org>
|
|
Fixes issue 757.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1064041
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1048041
|
|
Fixes issue 589.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1032044
|
|
R=adg
CC=golang-dev
http://codereview.appspot.com/1004045
|
|
R=gri, r, bflm
CC=golang-dev
http://codereview.appspot.com/1032043
|
|
- fixed a couple of bugs in the process
(shift right was incorrect for negative numbers)
- added more tests and made some tests more robust
- changed pidigits back to using shifts to multiply
by 2 instead of add
This improves pidigit -s -n 10000 by approx. 5%:
user 0m6.496s (old)
user 0m6.156s (new)
R=rsc
CC=golang-dev
http://codereview.appspot.com/963044
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/986044
|
|
Fixes issue 589.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1043042
|
|
Fixes issue 662.
R=ken2
CC=golang-dev
http://codereview.appspot.com/978043
|
|
Fixes issue 746.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1040042
|
|
Fixes issue 755.
R=ken2
CC=golang-dev
http://codereview.appspot.com/965047
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/962046
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1044041
|
|
This results in an improvement of > 35% for the existing Mul benchmark
using the same karatsuba threshold, and an improvement of > 50% with
a slightly higher threshold (32 instead of 30):
big.BenchmarkMul 500 6731846 ns/op (old alg.)
big.BenchmarkMul 500 4351122 ns/op (new alg.)
big.BenchmarkMul 500 3133782 ns/op (new alg., new theshold)
Also:
- tweaked calibrate.go, use same benchmark as for Mul benchmark
R=rsc
CC=golang-dev
http://codereview.appspot.com/1037041
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/974043
|
|
provide assembly versions (for x86-64 for now)
(Not yet used - waiting for previous CL to clear)
R=rsc
CC=golang-dev
http://codereview.appspot.com/1040041
|
|
R=rsc
CC=go-dev, golang-dev
http://codereview.appspot.com/1006045
|
|
R=rsc, Joe Poirier
CC=golang-dev
http://codereview.appspot.com/1015043
Committer: Russ Cox <rsc@golang.org>
|
|
Fixes issue 717.
R=adg, PeterGo
CC=golang-dev
http://codereview.appspot.com/1032041
|
|
R=rsc, brainman
CC=golang-dev
http://codereview.appspot.com/961047
Committer: Russ Cox <rsc@golang.org>
|
|
Fixes issue 749.
R=ken2
CC=golang-dev
http://codereview.appspot.com/963043
|
|
Fixes issue 745.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1008045
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1019043
Committer: Russ Cox <rsc@golang.org>
|
|
Fixes issue 686.
R=rsc
CC=adg, golang-dev
http://codereview.appspot.com/979044
Committer: Russ Cox <rsc@golang.org>
|
|
R=r, rsc
CC=golang-dev
http://codereview.appspot.com/946046
|
|
R=adg
CC=golang-dev
http://codereview.appspot.com/993042
Committer: Andrew Gerrand <adg@golang.org>
|
|
immediately
Also, calling fmt, ioutil and os "builtin" is probably confusing.
R=adg
CC=golang-dev
http://codereview.appspot.com/965045
Committer: Andrew Gerrand <adg@golang.org>
|
|
Fixes issue 549.
R=adg
CC=golang-dev
http://codereview.appspot.com/1019042
|
|
Fixes issue 686.
R=adg
CC=golang-dev
http://codereview.appspot.com/961046
|
|
Fixes issue 674.
R=adg
CC=golang-dev
http://codereview.appspot.com/1018042
|
|
inaccurate in meaning.
R=gri
CC=golang-dev
http://codereview.appspot.com/964045
|
|
R=r
CC=golang-dev, rsc
http://codereview.appspot.com/1014043
Committer: Rob Pike <r@golang.org>
|
|
earlier.
R=rsc
CC=golang-dev
http://codereview.appspot.com/972045
|
|
5l, 6l, 8l: change ELF header so that strip doesn't destroy binary
Fixes issue 261.
R=iant, r
CC=golang-dev
http://codereview.appspot.com/994044
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/972044
|
|
R=r, rsc
CC=golang-dev
http://codereview.appspot.com/963042
|
|
noticed this as I was submitting the previous CL :(
R=r
CC=golang-dev
http://codereview.appspot.com/989045
|
|
R=r, rsc, gri
CC=golang-dev
http://codereview.appspot.com/887045
|
|
Plus:
- calibration "test" - include in tests with gotest -calibrate
- basic Mul benchmark
- extra multiplication tests
- various cleanups
This change improves multiplication speed of numbers >= 30 words
in length (current threshold; found empirically with calibrate):
The multiplication benchmark (multiplication of a variety of long numbers)
improves by ~35%, individual multiplies can be significantly faster.
gotest -benchmarks=Mul
big.BenchmarkMul 500 6829290 ns/op (w/ Karatsuba)
big.BenchmarkMul 100 10600760 ns/op
There's no impact on pidigits for -n=10000 or -n=20000
because the operands are are too small.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1004042
Committer: Robert Griesemer <gri@golang.org>
|
|
Fixes issue 610.
R=adg
CC=golang-dev
http://codereview.appspot.com/1006044
|
|
Fixes issue 483.
R=dsymonds, dsymonds1
CC=golang-dev
http://codereview.appspot.com/965044
|
|
Fixes issue 750.
R=adg
CC=golang-dev
http://codereview.appspot.com/988043
|
|
Fixes issue 743.
R=r, rsc
CC=golang-dev
http://codereview.appspot.com/944044
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1013042
|
|
R=ken2
CC=golang-dev
http://codereview.appspot.com/944043
|