- 17 Nov, 2012 8 commits
-
-
Robert Griesemer authored
Fixes #4400. R=r, mirtchovski CC=golang-dev https://golang.org/cl/6782084
-
Shenghou Ma authored
R=dave, remyoudompheng CC=golang-dev https://golang.org/cl/6854056
-
Robert Griesemer authored
Replacement CL for 6813061; thanks to minux for prototyping. Fixes #4300. R=minux.ma CC=golang-dev https://golang.org/cl/6782082
-
Shenghou Ma authored
now we load jquery in its parent--godoc.html. R=golang-dev, adg CC=golang-dev https://golang.org/cl/6814112
-
Shenghou Ma authored
Write() can safely use uint32 loads when input is aligned. Also add test and benchmarks for unaligned writes. Benchmark result obtained by Dave Cheney on ARMv5TE @ 1.2GHz: benchmark old ns/op new ns/op delta BenchmarkHash8Bytes 4104 3417 -16.74% BenchmarkHash1K 22061 11208 -49.20% BenchmarkHash8K 146630 65148 -55.57% BenchmarkHash8BytesUnaligned 4128 3436 -16.76% BenchmarkHash1KUnaligned 22054 21473 -2.63% BenchmarkHash8KUnaligned 146658 146909 +0.17% benchmark old MB/s new MB/s speedup BenchmarkHash8Bytes 1.95 2.34 1.20x BenchmarkHash1K 46.42 91.36 1.97x BenchmarkHash8K 55.87 125.74 2.25x BenchmarkHash8BytesUnaligned 1.94 2.33 1.20x BenchmarkHash1KUnaligned 46.43 47.69 1.03x BenchmarkHash8KUnaligned 55.86 55.76 1.00x R=golang-dev, dave, bradfitz CC=golang-dev https://golang.org/cl/6782072
-
Shenghou Ma authored
also fix one out-of-date comment in cmd/ld/pe.c as well. R=golang-dev, alex.brainman CC=golang-dev https://golang.org/cl/6843061
-
Dave Cheney authored
Fixes #4393. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6854058
-
Robin Eklind authored
All of the errors were located using "go vet ./..." in "src/pkg". R=golang-dev, iant CC=golang-dev https://golang.org/cl/6856056
-
- 16 Nov, 2012 7 commits
-
-
Brad Fitzpatrick authored
R=golang-dev, r CC=golang-dev https://golang.org/cl/6847063
-
Robert Griesemer authored
ASTs may be created by various tools and built from nodes of different files. An incorrectly constructed AST will likely not print at all, but a (structurally) correct AST with bad position information should still print structurally correct. One heuristic used was to reset indentation when the filename in the position information of nodes changed. However, this can lead to wrong indentation for structurally correct ASTs. Fix: Don't change the indentation in this case. Related to issue 4300. R=r CC=golang-dev https://golang.org/cl/6849066
-
Robert Griesemer authored
ast.Walk needs to check for it or it will crash. R=r CC=golang-dev https://golang.org/cl/6852062
-
Dmitriy Vyukov authored
This significantly decreases amount of shadow memory mapped by race detector. I haven't tested on Windows, but on Linux it reduces virtual memory size from 1351m to 330m for fmt.test. Fixes #4379. R=golang-dev, alex.brainman, iant CC=golang-dev https://golang.org/cl/6849057
-
Oling Cat authored
R=golang-dev, dsymonds CC=golang-dev https://golang.org/cl/6843069
-
Joel Sing authored
Add support for loading X.509 key pairs that consist of a certificate with an EC public key and its corresponding EC private key. R=agl CC=golang-dev https://golang.org/cl/6776043
-
Dmitriy Vyukov authored
Reincarnation of https://golang.org/cl/6817086 (sent from another account). It is ugly because sync.Once will cause allocation of a closure. Fixes #4343. R=golang-dev, bradfitz, alex.brainman CC=golang-dev https://golang.org/cl/6856046
-
- 15 Nov, 2012 8 commits
-
-
Marcel van Lohuizen authored
compare incrementally. Also modified collation API to be more high-level by removing the need for an explicit buffer to be passed as an argument. This considerably speeds up Compare and CompareString. This change also eliminates the need to reinitialize the normalization buffer for each use of an iter. This also significantly improves performance for Key and KeyString. R=r, rsc CC=golang-dev https://golang.org/cl/6842050
-
Brad Fitzpatrick authored
API change. R=golang-dev, r, iant, rsc CC=golang-dev https://golang.org/cl/6844048
-
Andrew Gerrand authored
R=golang-dev, bradfitz, ftrvxmtrx, franciscossouza, r, minux.ma CC=golang-dev https://golang.org/cl/6843044
-
Rob Pike authored
Fixes #4386. R=golang-dev, iant CC=golang-dev https://golang.org/cl/6854050
-
Dmitriy Vyukov authored
R=golang-dev, iant CC=golang-dev https://golang.org/cl/6782075
-
Ian Lance Taylor authored
R=golang-dev, dave, jsing CC=golang-dev https://golang.org/cl/6847051
-
Dave Cheney authored
Revert to the shell builtin to avoid hosts that do not have /usr/bin/time. R=golang-dev, iant CC=golang-dev https://golang.org/cl/6848054
-
Dave Cheney authored
test/run.go already executes tests in parallel where possible. An unknown GOMAXPROCS value during the tests is known to cause failures with tests that measure allocations. ref: https://groups.google.com/d/topic/golang-nuts/tgMhFJ3F5WY/discussion R=fullung, minux.ma, r CC=golang-dev https://golang.org/cl/6847050
-
- 14 Nov, 2012 11 commits
-
-
Robin Eklind authored
R=gri CC=gobot, golang-dev https://golang.org/cl/6855048
-
Joel Sing authored
Add support for parsing SEC1 EC private keys and PKCS8 encapsulated EC private key structures. R=agl CC=golang-dev https://golang.org/cl/6767045
-
Joel Sing authored
Since we no longer skip the first entry when reading a symbol table, we no longer need to allow for the offset difference when processing the GNU version symbols. Unbreaks builds on Linux. R=golang-dev, agl, iant CC=golang-dev https://golang.org/cl/6843057
-
Joel Sing authored
Do not skip the first symbol in the symbol table. Any other indexes into the symbol table (for example, indexes in relocation entries) will now refer to the symbol following the one that was intended. Add an object that contains debug relocations, which debug/dwarf failed to decode correctly. Extend the relocation tests to cover this case. Note that the existing tests passed since the symbol following the symbol that required relocation is also of type STT_SECTION. Fixes #4107. R=golang-dev, mikioh.mikioh, iant, iant CC=golang-dev https://golang.org/cl/6848044
-
Dmitriy Vyukov authored
Currently race detector runtime just disables race detection in the finalizer goroutine. It has false positives when a finalizer writes to shared memory -- the race with finalizer is reported in a normal goroutine that accesses the same memory. After this change I am going to synchronize the finalizer goroutine with the rest of the world in racefingo(). This is closer to what happens in reality and so does not have false positives. And also add README file with instructions how to build the runtime. R=golang-dev, minux.ma, rsc CC=golang-dev https://golang.org/cl/6810095
-
Dmitriy Vyukov authored
It allows to catch e.g. a data race between atomic write and non-atomic write, or Mutex.Lock() and mutex overwrite (e.g. mu = Mutex{}). R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6817103
-
Dmitriy Vyukov authored
R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6821096
-
Dmitriy Vyukov authored
+ do not instrument go.itab.* R=golang-dev, rsc CC=golang-dev https://golang.org/cl/6819106
-
Andrew Gerrand authored
Update #4125 R=golang-dev, r CC=golang-dev https://golang.org/cl/6846045
-
Joel Sing authored
Regenerate z-files for OpenBSD/386 on OpenBSD 5.2. R=golang-dev, bradfitz, mikioh.mikioh CC=golang-dev https://golang.org/cl/6844044
-
Brad Fitzpatrick authored
This is a simplified version of earlier versions of this CL and now only fixes obviously incorrect things, without changing the locking on bodyEOFReader. I'd like to see if this is sufficient before changing the locking. Update #4191 R=golang-dev, rsc, dave CC=golang-dev https://golang.org/cl/6739055
-
- 13 Nov, 2012 6 commits
-
-
Robert Griesemer authored
The existing algorithm did not properly propagate the type count from one level to the next, and as a consequence it missed collisions. Properly propagate multiplicity (count) information to the next level. benchmark old ns/op new ns/op delta BenchmarkFieldByName1 182 180 -1.10% BenchmarkFieldByName2 6273 6183 -1.43% BenchmarkFieldByName3 49267 46784 -5.04% Fixes #4355. R=rsc CC=golang-dev https://golang.org/cl/6821094
-
Russ Cox authored
TBR=iant CC=golang-dev https://golang.org/cl/6843050
-
Russ Cox authored
In order to add these, we need to be able to find references to such types that already exist in the binary. To do that, introduce a new linker section holding a list of the types corresponding to arrays, chans, maps, and slices. To offset the storage cost of this list, and to simplify the code, remove the interface{} header from the representation of a runtime type. It was used in early versions of the code but was made obsolete by the kind field: a switch on kind is more efficient than a type switch. In the godoc binary, removing the interface{} header cuts two words from each of about 10,000 types. Adding back the list of pointers to array, chan, map, and slice types reintroduces one word for each of about 500 types. On a 64-bit machine, then, this CL *removes* a net 156 kB of read-only data from the binary. This CL does not include the needed support for precise garbage collection. I have created issue 4375 to track that. This CL also does not set the 'algorithm' - specifically the equality and copy functions - for a new array correctly, so I have unexported ArrayOf for now. That is also part of issue 4375. Fixes #2339. R=r, remyoudompheng, mirtchovski, iant CC=golang-dev https://golang.org/cl/6572043
-
Brad Fitzpatrick authored
Prep for future CL. R=gri CC=golang-dev https://golang.org/cl/6843048
-
Russ Cox authored
Incorporates code from CL 6828055. Fixes #2142. R=golang-dev, iant, devon.odell CC=golang-dev https://golang.org/cl/6826088
-
Joel Sing authored
Regenerate z-files for OpenBSD/amd64 on OpenBSD 5.2. R=golang-dev, bradfitz, mikioh.mikioh CC=golang-dev https://golang.org/cl/6843045
-