Age | Commit message (Collapse) | Author | Files | Lines |
|
Fixes issue 772.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1206043
|
|
- implemented setWord, use it where setUint64 is wrong
- divLarge: use fast mulWW, divWW; implemented mulWW, divWW
- better assembly code for addMulVVW
R=rsc
CC=golang-dev
http://codereview.appspot.com/1258042
|
|
TBR=r
CC=golang-dev
http://codereview.appspot.com/1230042
|
|
R=rsc, r
CC=golang-dev
http://codereview.appspot.com/946045
Committer: Russ Cox <rsc@golang.org>
|
|
Uses the SSE2 max, min instructions. Also shorter sqrt_amd64.s
R=rsc
CC=golang-dev
http://codereview.appspot.com/1216042
Committer: Russ Cox <rsc@golang.org>
|
|
IEEE 754 says: sqrt(-0) = -0
R=rsc
CC=golang-dev
http://codereview.appspot.com/1098041
Committer: Russ Cox <rsc@golang.org>
|
|
Fixes issue 784.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1196043
|
|
Cached string indexing in inner loop of Btoui64.
Before:
strconv_test.BenchmarkAtoi 5000000 309 ns/op
strconv_test.BenchmarkAtoiNeg 5000000 325 ns/op
strconv_test.BenchmarkAtoi64 5000000 465 ns/op
strconv_test.BenchmarkAtoi64Neg 5000000 469 ns/op
After:
strconv_test.BenchmarkAtoi 10000000 182 ns/op
strconv_test.BenchmarkAtoiNeg 10000000 193 ns/op
strconv_test.BenchmarkAtoi64 10000000 251 ns/op
strconv_test.BenchmarkAtoi64Neg 10000000 258 ns/op
R=golang-dev, gri
CC=golang-dev
http://codereview.appspot.com/1227042
Committer: Robert Griesemer <gri@golang.org>
|
|
- support for binary prefix 0b (to match fmt.Format)
- renamed nat.new -> nat.setUint64 for consistency
- more tests
R=r
CC=golang-dev
http://codereview.appspot.com/1233041
Committer: Robert Griesemer <gri@golang.org>
|
|
Also update range of Phase and Polar due to signed zero.
[Phase(cmplx(-1, +0)) = pi and Phase(cmplx(-1, -0)) = -pi]
R=rsc, r
CC=golang-dev
http://codereview.appspot.com/1235041
Committer: Rob Pike <r@golang.org>
|
|
Fixes issue 779.
R=r
CC=golang-dev
http://codereview.appspot.com/1218042
|
|
R=rsc, r
CC=golang-dev
http://codereview.appspot.com/1195041
Committer: Rob Pike <r@golang.org>
|
|
Fixes issue 775.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1180042
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1186041
|
|
Fixes issue 773.
R=adg
CC=golang-dev
http://codereview.appspot.com/1120042
|
|
Timings (as for change 1122043) go from 49ms to 48ms ish. It's
mostly lost in the noise, but it probably doesn't hurt.
R=r
CC=golang-dev
http://codereview.appspot.com/1179041
|
|
To draw.Draw a 32x32 image.Alpha 10000 times,
Before: 633ms with 10000 mallocs
After: 49ms with 0 mallocs
These times are just blitting an image.Alpha, and do not include
rasterizing a glyph's vector contours to an image.Alpha.
The "generic" test case in draw_test.go tests this fast path.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1122043
|
|
methods when looking up names.
Fixes issue 764.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1170041
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1173041
Committer: Russ Cox <rsc@golang.org>
|
|
- removed last argument (n) from all core arithmetic routines;
instead, use the length of the result
- simplified nat.make implementation and chose a better capacity
for new values, removed a TODO in the process
Changing the constant e from 1 (old) to 4 (new) improved
pidigits -s -n 10000 by ~9% (on a 3.06GHz Intel Core 2 Duo):
user 0m3.882s (old)
user 0m3.549s (new)
R=rsc
CC=golang-dev
http://codereview.appspot.com/1133043
|
|
- pass []Word instead of *Word to core arithmetic functions
- remove dead code
R=rsc
CC=golang-dev
http://codereview.appspot.com/1154042
|
|
NewDeflater -> NewWriter
NewInflater -> NewReader
Deflater -> Compressor
Inflater -> Decompressor
R=rsc
CC=golang-dev
http://codereview.appspot.com/1166041
|
|
returned to caller.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1164041
|
|
values are encoded for transmission.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1146041
|
|
This permits cgo callbacks to work when run in init code.
Otherwise cgocallback switches to the wrong stack address.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1123043
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1139042
|
|
- no need to make copies in cases of aliases
- removed deprecated internal shift functions
- minor unrelated simplifications
This change improves pidigits -s -n10000 by almost 20%:
user 0m6.156s (old)
user 0m4.999s (new)
(pidigits -s -n20000 goes from ~25s to ~19s)
R=rsc
CC=golang-dev
http://codereview.appspot.com/1149041
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1140041
|
|
fix a bug in the handling of indirect maps.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1132042
|
|
R=rsc, cw, nigeltao_golang
CC=golang-dev
http://codereview.appspot.com/1134041
|
|
Because maps are mostly a hidden type, they must be
implemented using reflection values and will not be as
efficient as arrays and slices.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1127041
|
|
R=golang-dev
CC=golang-dev
http://codereview.appspot.com/1121041
|
|
R=golang-dev, gri
CC=golang-dev
http://codereview.appspot.com/1072041
Committer: Robert Griesemer <gri@golang.org>
|
|
R=golang-dev, gri
CC=golang-dev
http://codereview.appspot.com/1113041
Committer: Robert Griesemer <gri@golang.org>
|
|
- renamed Len -> BitLen, simplified implementation
- renamed old Div, Mod, DivMod -> Que, Rem, QuoRem
- implemented Div, Mod, DivMod (Euclidian definition, more
useful in a mathematical context)
- fixed a bug in Exp (-0 was possible)
- added extra tests to check normalized results everywhere
- uniformly set Int.neg flag at the end of computations
- minor cosmetic cleanups
- ran all tests
R=rsc
CC=golang-dev
http://codereview.appspot.com/1091041
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1081042
|
|
R=gri
CC=golang-dev
http://codereview.appspot.com/1084041
|
|
Import _mulv from Inferno again, change R9 to R2.
Not sure what the other differences were for, but
they weren't working.
TBR=kaib
CC=golang-dev
http://codereview.appspot.com/1079041
|
|
R=gri
CC=golang-dev
http://codereview.appspot.com/987041
Committer: Robert Griesemer <gri@golang.org>
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1021043
|
|
Also improve the implementations of Equals and Compare.
R=rsc
CC=golang-dev
http://codereview.appspot.com/969047
|
|
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>
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/979043
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc
CC=golang-dev
http://codereview.appspot.com/1048041
|
|
Fixes issue 589.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1032044
|
|
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
|
|
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
|