- 08 Apr, 2015 16 commits
-
-
Shenghou Ma authored
Update #10373. Change-Id: I309e3df7608b9eef9339196fdc50dedf5f9439f5 Reviewed-on: https://go-review.googlesource.com/8452Reviewed-by:
Aram Hăvărneanu <aram@mgk.ro> Reviewed-by:
David Crawshaw <crawshaw@golang.org>
-
Shenghou Ma authored
Change-Id: I309e3df7608b9eef9339196fdc50dedf5f9439f4 Reviewed-on: https://go-review.googlesource.com/8451Reviewed-by:
-
Shenghou Ma authored
Change-Id: I309e3df7608b9eef9339196fdc50dedf5f9439f3 Reviewed-on: https://go-review.googlesource.com/8450Reviewed-by:
-
Shenghou Ma authored
Change-Id: I309e3df7608b9eef9339196fdc50dedf5f9439f2 Reviewed-on: https://go-review.googlesource.com/8439Reviewed-by:
-
Shenghou Ma authored
This CL introduces R_ADDRARM64, which is similar to R_ADDRPOWER. Fixes #10112. Change-Id: I309e3df7608b9eef9339196fdc50dedf5f9439f1 Reviewed-on: https://go-review.googlesource.com/8438Reviewed-by:
-
Shenghou Ma authored
Based on Michael Hudson-Doyle's patch: https://github.com/4ad/go/commit/b735215ee41b4237ec393d4669f6f55d4b27babc Change-Id: I309e3df7608b9eef9339196fdc50dedf5f9439f0 Reviewed-on: https://go-review.googlesource.com/8437Reviewed-by:
-
Shenghou Ma authored
Previously, the TestCompareAPI test would fail if runtime.Version() is "dev", or, more importantly, "go1.5"; because compareAPI depends on runtime.Version and -allow_new flag. Move that logic out make its test more robust. Change-Id: I8f40daa1838b8acd26adac8848762d95315053b0 Reviewed-on: https://go-review.googlesource.com/8622Reviewed-by:
Brad Fitzpatrick <bradfitz@golang.org>
-
Shenghou Ma authored
Based on cmd/9g/peep.go. Go 1 benchmark comparison: benchmark old ns/op new ns/op delta BenchmarkBinaryTree17 24328574000 18351639000 -24.57% BenchmarkFannkuch11 17029365000 10817758000 -36.48% BenchmarkFmtFprintfEmpty 291 223 -23.37% BenchmarkFmtFprintfString 1073 799 -25.54% BenchmarkFmtFprintfInt 1024 778 -24.02% BenchmarkFmtFprintfIntInt 1654 1277 -22.79% BenchmarkFmtFprintfPrefixedInt 1360 1083 -20.37% BenchmarkFmtFprintfFloat 2272 1415 -37.72% BenchmarkFmtManyArgs 5933 4742 -20.07% BenchmarkGobDecode 53166003 38584736 -27.43% BenchmarkGobEncode 37930156 30074874 -20.71% BenchmarkGzip 1880638900 1286832100 -31.57% BenchmarkGunzip 386343633 292194480 -24.37% BenchmarkHTTPClientServer 237077 179776 -24.17% BenchmarkJSONEncode 101731690 73116925 -28.13% BenchmarkJSONDecode 344655360 241277600 -29.99% BenchmarkMandelbrot200 28329778 12950809 -54.29% BenchmarkGoParse 21670755 16554244 -23.61% BenchmarkRegexpMatchEasy0_32 557 484 -13.11% BenchmarkRegexpMatchEasy0_1K 4687 4832 +3.09% BenchmarkRegexpMatchEasy1_32 539 483 -10.39% BenchmarkRegexpMatchEasy1_1K 5100 5080 -0.39% BenchmarkRegexpMatchMedium_32 796 651 -18.22% BenchmarkRegexpMatchMedium_1K 233099 182047 -21.90% BenchmarkRegexpMatchHard_32 13202 9897 -25.03% BenchmarkRegexpMatchHard_1K 401027 303602 -24.29% BenchmarkRevcomp 3837679666 2816546600 -26.61% BenchmarkTemplate 440608300 324831040 -26.28% BenchmarkTimeParse 1460 1019 -30.21% BenchmarkTimeFormat 1609 1174 -27.04% benchmark old MB/s new MB/s speedup BenchmarkGobDecode 14.44 19.89 1.38x BenchmarkGobEncode 20.24 25.52 1.26x BenchmarkGzip 10.32 15.08 1.46x BenchmarkGunzip 50.23 66.41 1.32x BenchmarkJSONEncode 19.07 26.54 1.39x BenchmarkJSONDecode 5.63 8.04 1.43x BenchmarkGoParse 2.67 3.50 1.31x BenchmarkRegexpMatchEasy0_32 57.38 66.05 1.15x BenchmarkRegexpMatchEasy0_1K 218.47 211.91 0.97x BenchmarkRegexpMatchEasy1_32 59.29 66.21 1.12x BenchmarkRegexpMatchEasy1_1K 200.76 201.54 1.00x BenchmarkRegexpMatchMedium_32 1.26 1.53 1.21x BenchmarkRegexpMatchMedium_1K 4.39 5.62 1.28x BenchmarkRegexpMatchHard_32 2.42 3.23 1.33x BenchmarkRegexpMatchHard_1K 2.55 3.37 1.32x BenchmarkRevcomp 66.23 90.24 1.36x BenchmarkTemplate 4.40 5.97 1.36x Fixes #10105. Change-Id: I353cc9fdf922e431821508c9dbbe4d9a85d64bd4 Signed-off-by:
Shenghou Ma <minux@golang.org> Reviewed-on: https://go-review.googlesource.com/8471Reviewed-by:
Dave Cheney <dave@cheney.net>
-
Shenghou Ma authored
Fixes #10369. Change-Id: If0a6d2b33c6862c9f7f862bdc997f2204072c6dc Reviewed-on: https://go-review.googlesource.com/8620 Run-TryBot: Minux Ma <minux@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
-
Shenghou Ma authored
Add test, and while we're at here, also add a test for ARM. Fixes #10343. Change-Id: Ic914df8233d4f1f495e2cc0743fbd37b7671bc91 Signed-off-by:
Rob Pike <r@golang.org>
-
Robert Griesemer authored
Change-Id: I58de76c49de6e43befb30b2bf677934e7952b5ab Reviewed-on: https://go-review.googlesource.com/8610Reviewed-by:
-
Robert Griesemer authored
The old code checked out a specific version of go/types from the x/tools repo. With go/types being part of the std repo, this is not necessary anymore. Also, for the same reason, the api tool is now built like any other regular command. There's no need to build it for each run. Removed the respective +build tags. Change-Id: I5088e4867223d676957084c24651ec05452ac495 Reviewed-on: https://go-review.googlesource.com/8564Reviewed-by:
-
Robert Griesemer authored
Temporary work-around so we can start using go/types in the std repo. Change-Id: I661465af791171b01cd23abf34dcb7eea6e26173 Reviewed-on: https://go-review.googlesource.com/8594Reviewed-by:
-
Robert Griesemer authored
This is a first step towards moving go/types from the tools repo into the std repo. The files were brought over via the added src/go/types.bash script for reproducability. The script can be removed once all dependencies on go/types have moved to the std repo go/types. The script moved packages as follows: - x/tools/go/types => go/types (type-checker) - x/tools/go/exact => go/exact (constants) - x/tools/go/gcimporter => go/types/internal/gcimporter The gcimporter is needed to be able to run tests. go/types should probably have some factory function to provide an appropriate importer. Some of the go/types tests fail for a handful of platforms (windows and nacl). In order to keep this change "clean" from manual changes, the next change will disable those tests for now so we can move forward. Change-Id: I448d8f7faa39ad2e04811911b699f7682627c224 Reviewed-on: https://go-review.googlesource.com/8530Reviewed-by:
-
Nigel Tao authored
Change-Id: I3621527c924a43724032f80a072505c60d929ab3 Reviewed-on: https://go-review.googlesource.com/8180Reviewed-by:
-
Alex Brainman authored
Fixes #9753 Change-Id: I6c641ed7ef4f687a108e7d937ab4b9c24d5baf5d Reviewed-on: https://go-review.googlesource.com/4940Reviewed-by:
-
- 07 Apr, 2015 24 commits
-
-
Robert Griesemer authored
Change-Id: Iff943d15e83e7db1f3c77f509a60e00ee2041d69 Reviewed-on: https://go-review.googlesource.com/8592Reviewed-by:
Robert Griesemer <gri@golang.org>
-
Robert Griesemer authored
For very out-of-range floating-point constants (1e100000000), precise formatting of the offending value for error messages is not needed and potentially extremely slow. This change resurrects an adjusted variant of the original code which uses float64 formatting in the common case (in-range values), and a fast manual approximation for out-of-range values. Change-Id: I2f6e53040929b8bf924dac4bb27c4d811ede48e2 Reviewed-on: https://go-review.googlesource.com/8470Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Michael Hudson-Doyle authored
Adds $DOLLAR to doc.go Change-Id: Ia0ccdd9b5292010297ae95debab4acf50d3c4d9c Reviewed-on: https://go-review.googlesource.com/8600Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
ALTree authored
Fixed bug that caused Exp(x, y, m) ( i.e. x**y (mod m) ) to return x instead of x (mod m) when y == 1. See issue page on github for more details. Added test case Fixes #9826 Change-Id: Ibabb58275a20c4231c9474199b7f1c10e54241ce Reviewed-on: https://go-review.googlesource.com/8409Reviewed-by:
-
Rob Pike authored
Fix the other places the slice length was being believed, and refactor the code to use a single function to unify the check. Fixes #10273. Change-Id: Ia62b25203fbe87c95d71a70ebc1db8d202eaa4a4 Reviewed-on: https://go-review.googlesource.com/8511Reviewed-by:
-
Dave Cheney authored
Add OGETG to the list of ignored operations. We don't instrument the runtime package, but calls to runtime.getg can appear in other packages, for example, after inlining runtime.LockOSThread. Change-Id: I8d6e91f1f3c8fd1302b596bdead42d588c059911 Reviewed-on: https://go-review.googlesource.com/8553Reviewed-by:
Dmitry Vyukov <dvyukov@google.com>
-
Dave Cheney authored
Add special case for OGETG which should never be instrumented. Change-Id: I7d082abb8608537f82b03362b687baf2a1d809dc Reviewed-on: https://go-review.googlesource.com/8551Reviewed-by:
-
Russ Cox authored
This more closely restores what the old C runtime did. (In C, g was an 'extern register' with the same effective implementation as in this CL.) On a late 2012 MacBookPro10,2, best of 5 old vs best of 5 new: benchmark old ns/op new ns/op delta BenchmarkBinaryTree17 4981312777 4463426605 -10.40% BenchmarkFannkuch11 3046495712 3006819428 -1.30% BenchmarkFmtFprintfEmpty 89.3 79.8 -10.64% BenchmarkFmtFprintfString 284 262 -7.75% BenchmarkFmtFprintfInt 282 262 -7.09% BenchmarkFmtFprintfIntInt 480 448 -6.67% BenchmarkFmtFprintfPrefixedInt 382 358 -6.28% BenchmarkFmtFprintfFloat 529 486 -8.13% BenchmarkFmtManyArgs 1849 1773 -4.11% BenchmarkGobDecode 12835963 11794385 -8.11% BenchmarkGobEncode 10527170 10288422 -2.27% BenchmarkGzip 436109569 438422516 +0.53% BenchmarkGunzip 110121663 109843648 -0.25% BenchmarkHTTPClientServer 81930 85446 +4.29% BenchmarkJSONEncode 24638574 24280603 -1.45% BenchmarkJSONDecode 93022423 85753546 -7.81% BenchmarkMandelbrot200 4703899 4735407 +0.67% BenchmarkGoParse 5319853 5086843 -4.38% BenchmarkRegexpMatchEasy0_32 151 151 +0.00% BenchmarkRegexpMatchEasy0_1K 452 453 +0.22% BenchmarkRegexpMatchEasy1_32 131 132 +0.76% BenchmarkRegexpMatchEasy1_1K 761 722 -5.12% BenchmarkRegexpMatchMedium_32 228 224 -1.75% BenchmarkRegexpMatchMedium_1K 63751 64296 +0.85% BenchmarkRegexpMatchHard_32 3188 3238 +1.57% BenchmarkRegexpMatchHard_1K 95396 96756 +1.43% BenchmarkRevcomp 661587262 687107364 +3.86% BenchmarkTemplate 108312598 104008540 -3.97% BenchmarkTimeParse 453 459 +1.32% BenchmarkTimeFormat 475 441 -7.16% The garbage benchmark from the benchmarks subrepo gets 2.6% faster as well. Change-Id: I320aeda332db81012688b26ffab23f6581c59cfa Reviewed-on: https://go-review.googlesource.com/8460Reviewed-by:
Rick Hudson <rlh@golang.org> Run-TryBot: Rick Hudson <rlh@golang.org> Reviewed-by:
Austin Clements <austin@google.com>
-
Mikio Hara authored
Using IPv6 link-local addresses to make connections between on-link nodes is useful for small distributed applications but it requires zone identifiers to distinguish a correct IP link. It's the same for transports using URI for destination discovery such as HTTP, WebSocket. This change allows Parse, ParseRequestURI functions and String method of URL to parse/return a literal IPv6 address followed by a zone identifier within a URI as described in RFC 6874. Fixes #6530. Change-Id: I2936ea65c1446994770cf2ee2c28a1c73faaa0ca Reviewed-on: https://go-review.googlesource.com/2431Reviewed-by:
-
kortschak authored
Change-Id: I6859bd9c9dba30fc5eeb9bbc1de90af67984944c Reviewed-on: https://go-review.googlesource.com/8526Reviewed-by:
-
Mikio Hara authored
This change deflakes TestDialerDualStackFDLeak, TestDialerDualStack, TestResolve{TCP,UDP,IP}Addr by removing external dependencies. Fixes #8764. Change-Id: I5cca0a93776cf05652e0e6a4a4ff4af392ccb885 Reviewed-on: https://go-review.googlesource.com/8485Reviewed-by: -
Alex Brainman authored
This reverts commit 9fa9f966. The change has broken darwin and netbsd builders. It needs to be tested properly. Change-Id: Id9e2d30caa8764c362c9f33890015dfc1aae0dab Reviewed-on: https://go-review.googlesource.com/8527 Run-TryBot: Alex Brainman <alex.brainman@gmail.com> Reviewed-by:
Alex Brainman <alex.brainman@gmail.com>
-
Brad Fitzpatrick authored
Change-Id: Iace8941c947253b1141f4194c5d2010c420ec220 Reviewed-on: https://go-review.googlesource.com/8540Reviewed-by:
-
Jiong Du authored
Bug Description: When reduce db.maxOpen via db.SetMaxOpenConns, the unnecssary connections won't been released until all other connections are free. Fixes #9453 Change-Id: I9afb2e4b184139b31029ae53d7f5fd1fdb8d8d7e Reviewed-on: https://go-review.googlesource.com/2200Reviewed-by:
-
Alex Brainman authored
Change-Id: I5939524f75f8fbbd67bd54b7c9e4b8c162337e9d Reviewed-on: https://go-review.googlesource.com/8525 Run-TryBot: Alex Brainman <alex.brainman@gmail.com> Reviewed-by:
-
Alex Brainman authored
Update #4339. Change-Id: Ic1a7535562b8b824ba166777725f7ba5b9623d77 Reviewed-on: https://go-review.googlesource.com/8523 Run-TryBot: Minux Ma <minux@golang.org> Reviewed-by:
-
Mikio Hara authored
This change tries to stop various tester goroutines at the end of each scope for avoiding interference between test cases including benchmarks. Not yet finished completely but enough to land upcoming changes to Dial functions. The rest will be fixed later. Change-Id: Ic38b8681a3a2ddbcd69ba3696f24a61d418a0346 Reviewed-on: https://go-review.googlesource.com/8398Reviewed-by:
-
Josh Bleecher Snyder authored
Convert Embedded, Method, and Colas to bools. I believe that this is the last of the Node fields that can be trivially converted to bools. No functional changes. Passes toolstash -cmp. Change-Id: I81962ee47866596341fc60d24d6959c20cd7fc1c Reviewed-on: https://go-review.googlesource.com/8440Reviewed-by:
-
Mikio Hara authored
Change-Id: I1f2d4e3b0351a7a47c3a6073833a17dbc0c7b05c Reviewed-on: https://go-review.googlesource.com/8520Reviewed-by:
-
Mikio Hara authored
This change adds testHookLookIP to enable to inject DNS name to IP address mappings for Happ{y,yish,ier} Eyeballs dial testing. Change-Id: I8ac04a594e1e2bd77909528df0552889914a7790 Reviewed-on: https://go-review.googlesource.com/8399Reviewed-by: -
Michael Hudson-Doyle authored
Ian complained about these in a review and then submitted the change before I could fix them. Change-Id: I23d890db2f3648ed1003ed3d13e7247435b913e5 Reviewed-on: https://go-review.googlesource.com/8480Reviewed-by:
-
Josh Bleecher Snyder authored
The tests in doc/progs appear to have been originally written for use with the old test driver. At some later point, they acquired their own test driver. Both ran tests in serial. This CL rewrites the current test driver in Go, runs tests concurrently, and cleans up historical artifacts from the old drivers. The primary motivation is to speed up all.bash. On my laptop, using tip, this CL reduces doc/progs test wall time from 26s to 7s. The savings will remain even when the compiler gets faster. Using Go 1.4, this CL reduces test wall time from 15s to 4s. Change-Id: Iae945a8490222beee76e8a2118a0d7956092f543 Reviewed-on: https://go-review.googlesource.com/8410Reviewed-by:
-
Michael Hudson-Doyle authored
A quick pass through link.go, mostly removing fields that are not used on the "creating a single object file" side of the fence. Change-Id: I35ba41378c2c418f7df2f2f88dce65bc64a1a45d Reviewed-on: https://go-review.googlesource.com/7672 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
-
Matthew Dempsky authored
Bison includes suggestions about what tokens are expected in the current state when there's only four or fewer of them. For example: syntax error: unexpected literal 2.01, expecting semicolon or newline or } This CL adds the same functionality to cmd/yacc, which fully restores the previous error message behavior from Go 1.4. Updates #9968. Change-Id: I2c1a1677c6d829a829d812c05e8813aa8829d09c Reviewed-on: https://go-review.googlesource.com/8494 Run-TryBot: Matthew Dempsky <mdempsky@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
