- 29 Aug, 2017 38 commits
-
-
Hiroshi Ioka authored
Change-Id: I8a54235c8b7bf1010f19d0d358cd4f76fc911d28 Reviewed-on: https://go-review.googlesource.com/59416Reviewed-by:
Ian Lance Taylor <iant@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Hiroshi Ioka authored
Change-Id: Ifc66bd8c32e788f35da3bd105bcdcfd04bd3dabc Reviewed-on: https://go-review.googlesource.com/59414Reviewed-by:
Ian Lance Taylor <iant@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org>
-
Hiroshi Ioka authored
* extract pkgname() and findlib() from the function for #18190. * rename const pkgname to const pkgdef to avoid confliction. Change-Id: Ie62509bfbddcf19cf92b5b12b598679a069e6e74 Reviewed-on: https://go-review.googlesource.com/59417Reviewed-by:
Ian Lance Taylor <iant@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Guilherme Rezende authored
Updates #21018 Change-Id: I00b6667fb5fee78559a391815f58760a2baea90e Reviewed-on: https://go-review.googlesource.com/50911Reviewed-by:
Ian Lance Taylor <iant@golang.org> Reviewed-by:
Avelino <t@avelino.xxx> Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Austin Clements authored
Right now we only kind of sort of trace GC STW events. We emit events around mark termination, but those start well after stopping the world and end before starting it again, and we don't emit any events for sweep termination. Fix this by generalizing EvGCScanStart/EvGCScanDone. These were already re-purposed to indicate mark termination (despite the names). This commit renames them to EvGCSTWStart/EvGCSTWDone, adds an argument to indicate the STW reason, and shuffles the runtime to generate them right before stopping the world and right after starting the world, respectively. These events will make it possible to generate precise minimum mutator utilization (MMU) graphs and could be useful in detecting non-preemptible goroutines (e.g., #20792). Change-Id: If95783f370781d8ef66addd94886028103a7c26f Reviewed-on: https://go-review.googlesource.com/55411Reviewed-by:
Rick Hudson <rlh@golang.org>
-
Guilherme Rezende authored
Also add more test cases. Change-Id: I53cc6484b25560fc7a4b5d44e73bbd9270c25769 Reviewed-on: https://go-review.googlesource.com/59950Reviewed-by:
Ian Lance Taylor <iant@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Daniel Martí authored
Found with mvdan.cc/unindent. It skipped the cases where parentheses would need to be added, where comments would have to be moved elsewhere, or where actions and simple logic would mix. One of them was of the form "err != nil && err == io.EOF", so the first part was removed. Change-Id: Ie504c2b03a2c87d10ecbca1b9270069be1171b91 Reviewed-on: https://go-review.googlesource.com/57690 Run-TryBot: Daniel Martí <mvdan@mvdan.cc> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Rob Pike authored
Some custom usage functions call it for clarity; others rely on the default behavior, which makes an explicit call redundant. Document that it's safe to be explicit. Fixes #21671. Change-Id: I08e9f47265582821cfd35995dff0c589cd85809d Reviewed-on: https://go-review.googlesource.com/59792Reviewed-by:
Dominik Honnef <dominik@honnef.co> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Hana Kim authored
Explaining how to enable tracing. Change-Id: Ic8391cb290742b4dc1efab15deba85853e182b4d Reviewed-on: https://go-review.googlesource.com/59612Reviewed-by:
Heschi Kreinick <heschi@google.com>
-
Ron Minnich authored
There are several distros now that no longer have /bin. Instead of assuming /bin/pwd, we will look for it in $PATH. Fixes #21684. Change-Id: I61478326500edeadc3c26803990550dad00c7971 Signed-off-by:
Ron Minnich <rminnich@gmail.com> Reviewed-on: https://go-review.googlesource.com/60010Reviewed-by:
Daniel Martí <mvdan@mvdan.cc> Run-TryBot: Daniel Martí <mvdan@mvdan.cc> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Elias Naur authored
CL 57291 broke on solaris because it depends on signal forwarding working for signals raised by dieFromSignal. Call sigtrampgo instead of sighandler directly, like the other unix platforms. Fixes the solaris builders. Change-Id: I6bf314c436d1edeaecc4b03f15a9155270919524 Reviewed-on: https://go-review.googlesource.com/59811 Run-TryBot: Elias Naur <elias.naur@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Hiroshi Ioka authored
Change-Id: Iec771d5bbdf510b6c5ec17a614da90e7974a6348 Reviewed-on: https://go-review.googlesource.com/59870Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Dmitri Shuralyov authored
Saying that they return t unchanged is misleading, because they return a modified t, stripped of any monotonic clock reading, as of Go 1.9. Fixes #21485. Change-Id: Icddf8813aed3d687fcefcd2fe542829438be6a0a Reviewed-on: https://go-review.googlesource.com/56690Reviewed-by:
Avelino <t@avelino.xxx> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
André Carvalho authored
This commit adds an example to the runtime/trace package on how to use the trace.Start and trace.Stop functions to trace the execution of a Go program and write its trace output to a file. Change-Id: Idf920398f1c3b9d185af9df5ce9293f2361db022 Reviewed-on: https://go-review.googlesource.com/51170Reviewed-by:
Hyang-Ah Hana Kim <hyangah@gmail.com>
-
Dmitry Vyukov authored
Change-Id: Icfb68e73ac38d0a0acc0cda1e41f9e9c5b75ecf5 Reviewed-on: https://go-review.googlesource.com/58110Reviewed-by:
Austin Clements <austin@google.com> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Matthew Dempsky authored
Updates #19683. Change-Id: I64b3b93a3ab14518a5376e1270bdd2a94bdd67ef Reviewed-on: https://go-review.googlesource.com/59611 Run-TryBot: Matthew Dempsky <mdempsky@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Robert Griesemer <gri@golang.org>
-
Marvin Stenger authored
This change could improve the hit rate on itabTable during growth. While we are here patch comments to refer to existing functions. Change-Id: I76f81c860a3d6107e077e7e3932550858a8b7651 Reviewed-on: https://go-review.googlesource.com/55912 Run-TryBot: Martin Möhrmann <moehrmann@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Keith Randall <khr@golang.org>
-
Carlos Eduardo Seo authored
This change adds new ppc64 instructions from the POWER9 ISA. This includes compares, loads, maths, register moves and the new random number generator and copy/paste facilities. Change-Id: Ife3720b90f5af184ff115bbcdcbce5c1302d39b6 Reviewed-on: https://go-review.googlesource.com/53930 Run-TryBot: Lynn Boger <laboger@linux.vnet.ibm.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Lynn Boger <laboger@linux.vnet.ibm.com>
-
Aliaksandr Valialkin authored
According to CL 31817, fmt cannot invoke String or Error methods on unexported struct fields. Fixes #17798. Change-Id: I0d516577298bc36daa9a94313c3874d64dc079e6 Reviewed-on: https://go-review.googlesource.com/44831Reviewed-by:
Rob Pike <r@golang.org>
-
Elias Naur authored
CL 49590 made it possible for external signal handlers to catch signals from a crashing Go process. This CL extends that support to handlers registered after the Go runtime has initialized. Updates #20392 (and possibly fix it). Change-Id: I18eccd5e958a505f4d1782a7fc51c16bd3a4ff9c Reviewed-on: https://go-review.googlesource.com/57291 Run-TryBot: Elias Naur <elias.naur@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Michael Munday authored
Fixes #21677. Change-Id: I869dee5f43df5d87d86922681726297e3024c562 Reviewed-on: https://go-review.googlesource.com/59810 Run-TryBot: Michael Munday <mike.munday@ibm.com> Reviewed-by:
Martin Möhrmann <moehrmann@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Alex Brainman authored
CL 59375 changed Reloc.Done to bool, but that change got lost in pe.go while merging. Restore that change. Change-Id: Ie5a89e85579cdc9282f504fefd56355cfeb49389 Reviewed-on: https://go-review.googlesource.com/59711 TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Alex Brainman <alex.brainman@gmail.com>
-
Manish Goregaokar authored
fixes #21635 Change-Id: I78716d47b6a930074e5394f7171d5f545dfba0bc Reviewed-on: https://go-review.googlesource.com/59690Reviewed-by:
Bryan Mills <bcmills@google.com> Reviewed-by:
Rob Pike <r@golang.org> Run-TryBot: Bryan Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Alex Brainman authored
Change-Id: I27b33f2425281bc1790528ae514d99a468ad7fce Reviewed-on: https://go-review.googlesource.com/59429Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I61f64287a488ed82e71639540697b6ade5a0426a Reviewed-on: https://go-review.googlesource.com/59428Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I1324b69ab9edb870589197fa601b5764634d42a4 Reviewed-on: https://go-review.googlesource.com/59427Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I506f5e146f3b5bf359d6932a85ac5572d3a3f103 Reviewed-on: https://go-review.googlesource.com/59426Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: If629b19cebc6ae8dfbf603dcd7f2dd3d0046a935 Reviewed-on: https://go-review.googlesource.com/59425Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I2da982fdef826aaecb86431f7d80ffc0fb2c4337 Reviewed-on: https://go-review.googlesource.com/59424Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I598e9da5587908f39faa13c11d2b42054f5a632d Reviewed-on: https://go-review.googlesource.com/59423Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I9fcae7bfd647b52d9b4f36f04710ba7921609c02 Reviewed-on: https://go-review.googlesource.com/59422Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I807412bd743d544195aaf9d720f81b128b4636b3 Reviewed-on: https://go-review.googlesource.com/59421Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I65058c150a19aa4e3083fd8754d08ea23721844b Reviewed-on: https://go-review.googlesource.com/59420Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I2458d1391af063d99ba1e446bcf12f7c41ae5f6b Reviewed-on: https://go-review.googlesource.com/59419Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Alex Brainman authored
Change-Id: I2bebee5566ee07786695f147c27661e69337a0f7 Reviewed-on: https://go-review.googlesource.com/59418Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Hiroshi Ioka authored
Without this CL, the system linker complains about absolute addressing in type..eqfunc.*. Updates #18190 Change-Id: I68db37a7f4c96b16a9c13baffc0f043a3048df6d Reviewed-on: https://go-review.googlesource.com/59373Reviewed-by:
Ian Lance Taylor <iant@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Kunpei Sakai authored
Fixes #21414 Change-Id: I8b25d416e6ead087b4a9965e8fa1a7f59ff07291 Reviewed-on: https://go-review.googlesource.com/59290Reviewed-by:
Hiroshi Ioka <hirochachacha@gmail.com> Reviewed-by:
Ian Lance Taylor <iant@golang.org> Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Hiroshi Ioka authored
Current code uses names like "x" and "s" which can conflict with user's code easily. Use cryptographic names. Fixes #21668 Change-Id: Ib6d3d6327aa5b92d95c71503d42e3a79d96c8e15 Reviewed-on: https://go-review.googlesource.com/59710Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
- 28 Aug, 2017 2 commits
-
-
Matthew Dempsky authored
Passes toolstash-check. Change-Id: I01ed1c04be5a23756742d461f13f1e587ea7ecb8 Reviewed-on: https://go-review.googlesource.com/59610 Run-TryBot: Matthew Dempsky <mdempsky@google.com> Reviewed-by:
Josh Bleecher Snyder <josharian@gmail.com>
-
Josh Bleecher Snyder authored
Passes toolstash-check. Change-Id: Id62bacff13fbd30de62b925d97a4e7bee1c66120 Reviewed-on: https://go-review.googlesource.com/59331 Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by:
Matthew Dempsky <mdempsky@google.com>
-