Age | Commit message (Collapse) | Author | Files | Lines |
|
R=rsc, r, gri
CC=golang-dev
http://codereview.appspot.com/1239043
Committer: Robert Griesemer <gri@golang.org>
|
|
Time to draw.Draw a 200x200 image fell from 18.4ms (and 1 malloc) to
5.6ms (and 0 mallocs). It's still relatively slow since it assumes
nothing about the src or mask images, but it does remove the malloc.
There are existing faster, more specialized paths for copies, fills
and image glyph masks.
Also added a "compare to a slow but obviously correct implementation"
check to draw_test.go.
R=rsc, r
CC=golang-dev
http://codereview.appspot.com/1223044
|
|
R=rsc
CC=adg, golang-dev, r
http://codereview.appspot.com/1253043
Committer: Russ Cox <rsc@golang.org>
|
|
R=rsc
CC=adg, golang-dev, r
http://codereview.appspot.com/1257041
Committer: Russ Cox <rsc@golang.org>
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1083041
|
|
R=rsc, r
CC=golang-dev
http://codereview.appspot.com/1103041
Committer: Russ Cox <rsc@golang.org>
|
|
Needed for CL 1103041 and beyond.
R=adg, bradfitzpatrick
CC=bradfitz, golang-dev
http://codereview.appspot.com/1172042
|
|
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>
|
|
Previously the compiler would just emit "internal compiler error" when
trying to compute the remainder of floats or complex types.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1243041
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>
|
|
has no access to yylex)
R=ken2, ken3
CC=golang-dev
http://codereview.appspot.com/813047
Committer: Ken Thompson <ken@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>
|
|
R=rsc, r
CC=golang-dev
http://codereview.appspot.com/1231041
Committer: Rob Pike <r@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>
|
|
R=adg
CC=golang-dev
http://codereview.appspot.com/1228041
|
|
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=ken2
CC=golang-dev
http://codereview.appspot.com/1138041
|
|
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>
|
|
external variable assignements.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1094041
|
|
- 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
|
|
Fixes issue 692.
R=ken2
CC=golang-dev
http://codereview.appspot.com/1092041
|
|
R=r
CC=golang-dev
http://codereview.appspot.com/1081042
|