- 05 Jun, 2012 2 commits
-
-
Nigel Tao authored
There already is a "Phase 6" a few lines above. R=rsc CC=golang-dev https://golang.org/cl/6270054
-
Rob Pike authored
R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6272050
-
- 04 Jun, 2012 22 commits
-
-
Rémy Oudompheng authored
It is not necessary for the test to be effective and uses a lot of resources in the compiler. Memory usage is halved and compilation around 8x faster. R=golang-dev, r, rsc, r CC=golang-dev https://golang.org/cl/6290044
-
Robert Griesemer authored
Fixes #3635. R=rsc, r, iant, kevlar, iant CC=golang-dev https://golang.org/cl/6217045
-
Russ Cox authored
The original implementation of closures created the underlying top-level function during walk, which is fairly late in the compilation process and caused ordering-based complications due to earlier stages that had to be repeated any number of times. Create the underlying function during typecheck, much earlier, so that later stages can be run just once. The result is a simpler compilation sequence. R=ken2 CC=golang-dev https://golang.org/cl/6279049
-
Shenghou Ma authored
although the comment says it uses libc's getenv, without NOPLAN9DEFINES it actually uses p9getenv which strdups. R=golang-dev, dave, rsc CC=golang-dev https://golang.org/cl/6285046
-
Russ Cox authored
The recent shuffle in parsing formats exposed probably unintentional behavior in time.Parse, namely that it was mostly ignoring ".99999" in the format, producing the following behavior: fmt.Println(time.Parse("03:04:05.999 MST", "12:00:00.888 PDT")) // error (.888 unexpected) fmt.Println(time.Parse("03:04:05.999", "12:00:00")) // error (input too short) fmt.Println(time.Parse("03:04:05.999 MST", "12:00:00 PDT")) // ok (extra bytes on input make it ok) http://play.golang.org/p/ESJ1UYXzq2 API CHANGE: This CL makes all three examples valid: ".999" can match an empty string or else a fractional second with at most nine digits. Fixes #3701. R=r, r CC=golang-dev https://golang.org/cl/6267045
-
Robert Griesemer authored
An attempt to profit from CL 6176043 (fix to superpolinomial runtime of karatsuba multiplication) and determine a better karatsuba threshold. The result indicates that 32 is still a reasonable value. Left the threshold as is (== 32), but made some minor changes to the calibrate code which are worthwhile saving (use of existing benchmarking code for better results, better use of package time). R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6260062
-
Marcel van Lohuizen authored
Regenerated tables using maketables. R=r, rsc CC=golang-dev https://golang.org/cl/6248067
-
Shenghou Ma authored
We can't depend on init() order, and certainly we don't want to register all future benchmarks that use jsonbytes or jsondata to init() in json_test.go, so we use a more general solution: make generation of jsonbytes and jsondata their own function so that the compiler will take care of the order. R=golang-dev, dave, rsc CC=golang-dev https://golang.org/cl/6282046
-
Joel Sing authored
Fix botched cgo_netbsd.go merge and remove redundant +build directive. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6282048
-
Joel Sing authored
R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6261056
-
Joel Sing authored
Split stdout/stderr into a separate file so that can be handled differently on some platforms. Both NetBSD and OpenBSD have defines for stdout/stderr that require some coercion in order for cgo to handle them correctly. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6247062
-
Benny Siegert authored
pipe2 is equivalent to pipe with flags set to 0. However, pipe2 was only added recently. Using pipe instead improves compatibility with NetBSD 5. R=jsing CC=golang-dev https://golang.org/cl/6268045
-
Brad Fitzpatrick authored
The new BenchmarkServer avoids profiling the client code by running it in a child process. R=rsc CC=golang-dev https://golang.org/cl/6260053
-
Andrew Gerrand authored
Fixes #3634. R=golang-dev, bsiegert, rsc CC=golang-dev https://golang.org/cl/6270047
-
Russ Cox authored
R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6273050
-
Ryan Barrett authored
specifically, adds a go-test element to compilation-error-regexp-alist[-alist]. Fixes #3629. R=golang-dev, rsc, sameer CC=golang-dev, jba https://golang.org/cl/6197091
-
Brad Fitzpatrick authored
The cleanup also makes it ~5% faster, but that's not the point of this CL. Optimizations can come in future CLs. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6286043
-
Brad Fitzpatrick authored
Removes code. No need for atoi helper. R=golang-dev, adg CC=golang-dev https://golang.org/cl/6285045
-
Nigel Tao authored
R=rsc CC=golang-dev https://golang.org/cl/6249071
-
Shenghou Ma authored
It's very unfortunate that the type of Data field of struct RawSockaddr is [14]uint8 on Linux/ARM instead of [14]int8 on all the others. btw, it should be [14]int8 according to my header files. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6275050
-
Brad Fitzpatrick authored
No need to fix case of "Host" string literal. R=golang-dev, dsymonds CC=golang-dev https://golang.org/cl/6278049
-
Mikio Hara authored
Make it possible to run test over symlinked GOROOT. R=golang-dev, r CC=golang-dev https://golang.org/cl/6268046
-
- 03 Jun, 2012 7 commits
-
-
Russ Cox authored
benchmark old ns/op new ns/op delta BenchmarkFormat 2495 937 -62.44% BenchmarkFormatNow 2308 889 -61.48% Update #3679. R=r CC=golang-dev https://golang.org/cl/6278047
-
Joel Sing authored
Move address info flags to per-platform files. This is needed to enable cgo on NetBSD (and later OpenBSD), as some of the currently used AI_* defines do not exist on these platforms. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6250075
-
Alex Brainman authored
Fixes #3685. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6261053
-
Russ Cox authored
TBR=lvd CC=golang-dev https://golang.org/cl/6275048
-
Russ Cox authored
R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6278045
-
Luuk van Dijk authored
R=rsc, ality, rogpeppe, minux.ma, dave CC=golang-dev https://golang.org/cl/5966075
-
Russ Cox authored
Use perfect cuckoo hash, to avoid binary search. Define Atom bits as offset+len in long string instead of enumeration, to avoid string headers. Before: 1909 string bytes + 6060 tables = 7969 total data After: 1406 string bytes + 2048 tables = 3454 total data benchmark old ns/op new ns/op delta BenchmarkLookup 83878 64681 -22.89% R=nigeltao, r CC=golang-dev https://golang.org/cl/6262051
-
- 02 Jun, 2012 9 commits
-
-
Shenghou Ma authored
16-bit uid calls are not always supported in newer kernels. R=dave, rsc, bradfitz CC=golang-dev https://golang.org/cl/6279047
-
Shenghou Ma authored
R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6270045
-
Shenghou Ma authored
Supported in ARMv5 and above. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6284043
-
Jan Ziak authored
R=rsc CC=golang-dev https://golang.org/cl/6255074
-
Charles L. Dorian authored
Ceil to 4.81 from 20.6 ns/op Floor to 4.37 from 13.5 ns/op Trunc to 3.97 from 14.3 ns/op Also changed three MOVSDs to MOVAPDs in log_amd64.s R=rsc, golang-dev CC=golang-dev https://golang.org/cl/6262048
-
Jan Mercl authored
Currently walk() doesn't check for err == SkipDir when iterating a directory list, but such promise is made in the docs for WalkFunc. Fixes #3486. R=rsc, r CC=golang-dev https://golang.org/cl/6257059
-
Shenghou Ma authored
R=dave, rsc CC=golang-dev https://golang.org/cl/6248070
-
Brad Fitzpatrick authored
Now that gri has made go/parser 15% faster, I offer this change to slow back down cmd/api ~proportionately, adding FreeBSD to the go1-checked set of platforms. Really we should have done this earlier. This will prevent us from breaking FreeBSD compatibility accidentally in the future. R=golang-dev, r CC=golang-dev https://golang.org/cl/6279044
-
Rob Pike authored
To avoid goroutines during init, the nextItem function was a clever workaround. Now that init goroutines are permitted, restore the original, simpler design. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6282043
-