- 06 Feb, 2015 21 commits
-
-
Josh Bleecher Snyder authored
eqstring does not need to check the length of the strings. Other architectures were done in a separate commit. While we're here, add a pointer equality check. Change-Id: Id2c8616a03a7da7037c1e9ccd56a549fc952bd98 Reviewed-on: https://go-review.googlesource.com/3956Reviewed-by:
Keith Randall <khr@golang.org>
-
Josh Bleecher Snyder authored
eqstring does not need to check the length of the strings. 6g benchmark old ns/op new ns/op delta BenchmarkCompareStringEqual 7.03 6.14 -12.66% BenchmarkCompareStringIdentical 3.36 3.04 -9.52% 5g benchmark old ns/op new ns/op delta BenchmarkCompareStringEqual 238 232 -2.52% BenchmarkCompareStringIdentical 90.8 80.7 -11.12% The equivalent PPC changes are in a separate commit because I don't have the hardware to test them. Change-Id: I292874324b9bbd9d24f57a390cfff8b550cdd53c Reviewed-on: https://go-review.googlesource.com/3955Reviewed-by:
Keith Randall <khr@golang.org>
-
Robert Griesemer authored
- clarified representation of +/-Inf - only 0 and Inf values can have 0 precision - a zero precision value used as result value takes the max precision of the arguments (to be fine-tuned for setters) - the zero precision approach makes Float zero values possible (they represent +0) - more tests Missing: Filling in the blanks. More tests. Change-Id: Ibb4f97e12e1f356c3085ce80f3464e97b82ac130 Reviewed-on: https://go-review.googlesource.com/4000Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Péter Surányi authored
Only documentation / comment changes. Update references to point to golang.org permalinks or go.googlesource.com/go. References in historical release notes under doc are left as is. Change-Id: Icfc14e4998723e2c2d48f9877a91c5abef6794ea Reviewed-on: https://go-review.googlesource.com/4060Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Shenghou Ma authored
Change-Id: I40feb9e65615a0f683cfc3f54e0c0cfabcf6a787 Reviewed-on: https://go-review.googlesource.com/4010Reviewed-by:
Minux Ma <minux@golang.org>
-
Mikio Hara authored
And the silence of the git-codereview. Change-Id: If3f7fe2de2ab4c1756f3cef8267199049d468b31 Reviewed-on: https://go-review.googlesource.com/3983Reviewed-by:
Minux Ma <minux@golang.org>
-
Shenghou Ma authored
Fixes #837. Change-Id: I2d601504addbd220b304d32d587144d2a702f753 Reviewed-on: https://go-review.googlesource.com/2127Reviewed-by:
David Crawshaw <crawshaw@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Shenghou Ma authored
cmd/dist: recognize darwin/arm as (host) goos/goarches. also hard code GOARM=7 for darwin/arm. make.bash: don't pass -mmacosx-version-min=10.6 when building for darwin/arm. Change-Id: If0ecd84a5179cd9bb61b801ac1899adc45f12f75 Reviewed-on: https://go-review.googlesource.com/2126Reviewed-by:
Ian Lance Taylor <iant@golang.org> Reviewed-by:
David Crawshaw <crawshaw@golang.org>
-
Shenghou Ma authored
On Darwin/ARM, because libSystem doesn't provide functions for __sync_fetch_and_add, and only clang can inline that function, skip the test when building with GCC. Change-Id: Id5e9d8f9bbe1e6bcb2f381f0f66cf68aa95277c7 Reviewed-on: https://go-review.googlesource.com/2125Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Shenghou Ma authored
Change-Id: I936be12eed95b99d1d20ed16fb785182e1817b33 Reviewed-on: https://go-review.googlesource.com/2124Reviewed-by:
David Crawshaw <crawshaw@golang.org>
-
Shenghou Ma authored
Change-Id: Id6f7fa12084204bc3a200f423c7966ce2a0b63a0 Reviewed-on: https://go-review.googlesource.com/2123Reviewed-by:
Ian Lance Taylor <iant@golang.org> Reviewed-by:
David Crawshaw <crawshaw@golang.org>
-
Shenghou Ma authored
Change-Id: I213a8ab0b8c027a7b73567aeefdca73fd10eae28 Reviewed-on: https://go-review.googlesource.com/2122Reviewed-by:
David Crawshaw <crawshaw@golang.org>
-
Shenghou Ma authored
Change-Id: I63110daad2d62ae72ab1f33a40464d76e6205627 Reviewed-on: https://go-review.googlesource.com/2121Reviewed-by:
David Crawshaw <crawshaw@golang.org>
-
Shenghou Ma authored
liblink: - set dummy value for ctxt->tlsoffset. cmd/ld: - always do external linking when using cgo on darwin/arm, as our linker might not generate codesign-compatible binary. cmd/5l: - support generate ARM Mach-O binaries - add machoreloc1() that translate our internal relocation to macho relocations used by external linking. Change-Id: Ic5454aeb87009aaf8f1453ec7fe33e6da55d5f06 Reviewed-on: https://go-review.googlesource.com/3273Reviewed-by:
David Crawshaw <crawshaw@golang.org>
-
Shenghou Ma authored
Libc on Darwin/ARM has a buggy implementation of ldexp and frexp that could not handle denormal numbers. Also disable VFP runfast (flush-to-zero) mode so that the gc compiler can correctly handle denormal constants used in math and strconv tests. Change-Id: Ie64220b882f414e0b37f406f38181c3586104d46 Reviewed-on: https://go-review.googlesource.com/2119Reviewed-by:
David Crawshaw <crawshaw@golang.org> Reviewed-by:
Dave Cheney <dave@cheney.net>
-
Shenghou Ma authored
In the old code, liblink, cmd/ld and runtime all have code determine whether runtime.tlsg is an actual variable or a placeholder for TLS relocation. This change consolidate them into one: the runtime/tls_arm.s will ultimately determine the type of that variable. Change-Id: I3b3f80791a1db4c2b7318f81a115972cd2237e43 Reviewed-on: https://go-review.googlesource.com/2118Reviewed-by:
David Crawshaw <crawshaw@golang.org> Reviewed-by:
Dave Cheney <dave@cheney.net>
-
Shenghou Ma authored
Fixes #9742. Change-Id: Ifedf7ff9465bc49534b708d414c8e435ee9ce6cd Reviewed-on: https://go-review.googlesource.com/3970Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Rahul Chaudhry authored
The main issue is that the misc/cgo/{stdio,life} tests are silently getting skipped when invoked from run.bash. run.go should ignore any build tags after the first blank line in source file. It already checks for test actions only upto the first blank line. Build tags must be specified in the same block. See http://golang.org/cl/3675 for background. Change-Id: Id8abf000119e3335f7250d8ef34aac7811fc9dff Reviewed-on: https://go-review.googlesource.com/3812Reviewed-by:
Minux Ma <minux@golang.org>
-
Rahul Chaudhry authored
issue9355 generated a file a.[568] in test/ directory and left it there. For tests like these, it is best to chdir to a test specific directory before generating any temporary files, since the tests are running in parallel and might otherwise race with each other for the same files. Change-Id: I58d96256d4d8ee3fda70d81077f19006064a7425 Reviewed-on: https://go-review.googlesource.com/3813Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Andrew Ekstedt authored
PalettedColorModel was renamed to color.Palette over three years ago by https://golang.org/cl/5132048. Change-Id: I0204ade10eabff45620fda2990fed428c65d871e Reviewed-on: https://go-review.googlesource.com/3305Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Nigel Tao <nigeltao@golang.org>
-
Nigel Tao authored
and empty keys. Also reject malformed (quoted) values. See also https://go-review.googlesource.com/3952 Change-Id: Ice6de33b09f9904b28e410a680a90aa6c8c76fed Reviewed-on: https://go-review.googlesource.com/3953Reviewed-by:
Rob Pike <r@golang.org>
-
- 05 Feb, 2015 12 commits
-
-
Robert Griesemer authored
Change-Id: I7514e643a6a81ca715adbf6f8d19d3b2dd43fe6c Reviewed-on: https://go-review.googlesource.com/3810Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Robert Griesemer authored
Change-Id: Ica419a1215ca33dc1cff1e9e4137f204591e3cee Reviewed-on: https://go-review.googlesource.com/3942Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Robert Griesemer authored
Change-Id: Ia30886569141ca2e0321bea6ee1d5c9e0f79d6f9 Reviewed-on: https://go-review.googlesource.com/3941Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Robert Griesemer authored
Change-Id: I4329c44b829fcd77e4f1a1d45904f0f8a280a595 Reviewed-on: https://go-review.googlesource.com/3940Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Erik Aigner authored
Change-Id: Ida064ff422d077d508ef94f52fc813daa6277a99 Reviewed-on: https://go-review.googlesource.com/3933Reviewed-by:
Brad Fitzpatrick <bradfitz@golang.org>
-
Russ Cox authored
This was working when 5a was built on x86 because REG_R0 = 32, and a 32-bit shift on x86 uses only the low 32 bits of the shift count. On ARM, the shift clamping is different. Moving to Go will avoid these differing shift semantics. I tripped over and fixed this bug in new5a the same way earlier tonight. Change-Id: Id56aa0bb1830ccf250960f843e0acb8a0409e87d Reviewed-on: https://go-review.googlesource.com/3961Reviewed-by:
Dave Cheney <dave@cheney.net>
-
Dmitry Vyukov authored
Change-Id: I6ee7aa76673e51576b5a84c512b22a0f8af5b02f Reviewed-on: https://go-review.googlesource.com/3966Reviewed-by:
Dmitry Vyukov <dvyukov@google.com>
-
Evan Phoenix authored
These benchmarks are only for functions commonly used in loops. The other functions are typically used for inspection or setup and thus are not performance sensitive. Change-Id: I8d0a0ba2d8234ecacb40fa3aa9077bf93c8fe89c Reviewed-on: https://go-review.googlesource.com/3680Reviewed-by:
Dmitry Vyukov <dvyukov@google.com>
-
Russ Cox authored
This reverts commit da4abda2. Change-Id: Ifd09b3dae0af0c7cef5fbbc332c63e78dc90d6b1 Reviewed-on: https://go-review.googlesource.com/3960Reviewed-by:
Russ Cox <rsc@golang.org>
-
Shenghou Ma authored
Fixes #9756. Change-Id: If4ee6fe10f8f90294ff9c5e7480371494094b111 Signed-off-by:
Shenghou Ma <minux@golang.org> Reviewed-on: https://go-review.googlesource.com/3740Reviewed-by:
Ian Lance Taylor <iant@golang.org> Reviewed-by:
Alex Brainman <alex.brainman@gmail.com>
-
Shenghou Ma authored
Fixes #9759. Change-Id: I263f1251b9401371231374551c4f71c70cb6e359 Signed-off-by:
Shenghou Ma <minux@golang.org> Reviewed-on: https://go-review.googlesource.com/3931Reviewed-by:
Dave Cheney <dave@cheney.net>
-
Rob Pike authored
Followup to https://go-review.googlesource.com/3910 We only need 1000 iteratinons. Change-Id: Ib63ae53105176abec77bad9609d638aeda7bcd61 Reviewed-on: https://go-review.googlesource.com/3901Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
- 04 Feb, 2015 7 commits
-
-
Robert Griesemer authored
Change-Id: I10efa3bc8bc7f41100feabe17837f805a42d7eb6 Reviewed-on: https://go-review.googlesource.com/3842Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Robert Griesemer authored
(For zero values the strconv %b format prints the bias-adjusted exponent; there's no bias in Float.) Change-Id: I6f4dda9c3a50d02eac375cfe2c927c1540aae865 Reviewed-on: https://go-review.googlesource.com/3841Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Robert Griesemer authored
Change-Id: Id508ca2f6c087861e8c6bc536bc39e54dce09825 Reviewed-on: https://go-review.googlesource.com/3840Reviewed-by:
Alan Donovan <adonovan@google.com>
-
Hyang-Ah Hana Kim authored
In android-L, logging is done through the logd daemon. If logd daemon is available, send logging to logd. Otherwise, fallback to the legacy mechanism (/dev/log files). This change adds access/socket/connect calls to interact with the logd. Fixes golang/go#9398. Change-Id: I3c52b81b451f5862107d7c675f799fc85548486d Reviewed-on: https://go-review.googlesource.com/3350Reviewed-by:
David Crawshaw <crawshaw@golang.org>
-
Josh Bleecher Snyder authored
Change-Id: I00f18a8c62a8d63822d7e07fee1ac5f088cc9587 Reviewed-on: https://go-review.googlesource.com/3866Reviewed-by:
Josh Bleecher Snyder <josharian@gmail.com>
-
Russ Cox authored
This reverts commit 80590711. Conflicts: src/runtime/proc1.go (resolved by hand) Change-Id: Ic62e3e802f0007ff9012b37bc5fd41fcafc153d0 Reviewed-on: https://go-review.googlesource.com/3885Reviewed-by:
Dmitry Vyukov <dvyukov@google.com>
-
Russ Cox authored
Add Addr-checking for all Progs on input to liblink, in liblink/pass.c, including requiring use of TYPE_ADDR, not TYPE_CONST. Update compilers and assemblers to satisfy checks. Change-Id: Idac36b9f6805f0451cb541d2338992ca5eaf3963 Reviewed-on: https://go-review.googlesource.com/3801Reviewed-by:
Austin Clements <austin@google.com>
-