- 17 Mar, 2015 16 commits
-
-
David Crawshaw authored
The Go builders (and standard development cycle) for programs on iOS require running the programs under lldb. Unfortunately lldb intercepts SIGSEGV and will not give it back. https://llvm.org/bugs/show_bug.cgi?id=22868 We get around this by never letting lldb see the SIGSEGV. On darwin, Unix signals are emulated on top of mach exceptions. The debugger registers a task-level mach exception handler. We register a thread-level exception handler which acts as a faux signal handler. The thread-level handler gets precedence over the task-level handler, so we can turn the exception EXC_BAD_ACCESS into a panic before lldb can see it. Fixes #10043 Change-Id: I64d7c310dfa7ecf60eb1e59f094966520d473335 Reviewed-on: https://go-review.googlesource.com/7072Reviewed-by:
Minux Ma <minux@golang.org> Run-TryBot: David Crawshaw <crawshaw@golang.org>
-
Dave Cheney authored
Fix recover4.go to work on 64kb systems. Change-Id: I211cb048de1268a8bbac77c6f3a1e0b8c8277594 Reviewed-on: https://go-review.googlesource.com/7673Reviewed-by:
Minux Ma <minux@golang.org>
-
Jeremy Jackins authored
Change-Id: I367b5a837844e3bee1576c59497d37f5e67c761d Reviewed-on: https://go-review.googlesource.com/7674Reviewed-by:
Minux Ma <minux@golang.org>
-
Dave Cheney authored
This reverts commit 1313e798. Change-Id: I96cc58baf71156fdfbf8fd61332744bcc3ea52e5 Reviewed-on: https://go-review.googlesource.com/7670Reviewed-by:
Dave Cheney <dave@cheney.net>
-
Dave Cheney authored
Updates #10180 Temporarily disable this test on ppc64 systems as all our builders use 64k page size. We need a portable way to get the page size of the host so we can correctly size the mmap hole. Change-Id: Ibd36ebe2f54cf75a44667e2070c385f0daaca481 Reviewed-on: https://go-review.googlesource.com/7652Reviewed-by:
Andrew Gerrand <adg@golang.org>
-
Austin Clements authored
When checkmark fails, greyobject dumps both the object that pointed to the unmarked object and the unmarked object. This code cluttered up greyobject, was copy-pasted for the two objects, and the copy for dumping the unmarked object was not entirely correct. Extract object dumping out to a new function. This declutters greyobject and fixes the bugs in dumping the unmarked object. The new function is slightly cleaned up from the original code to have more natural control flow and shows a marker on the field in the base object that points to the unmarked object to make it easy to find. Change-Id: Ib51318a943f50b0b99995f0941d03ee8876b9fcf Reviewed-on: https://go-review.googlesource.com/7506Reviewed-by:
Rick Hudson <rlh@golang.org>
-
Austin Clements authored
scanobject no longer returns the new wbuf. Change-Id: I0da335ae5cd7ef7ea0e0fa965cf0e9f3a650d0e6 Reviewed-on: https://go-review.googlesource.com/7505Reviewed-by:
Rick Hudson <rlh@golang.org>
-
Russ Cox authored
This directory is processed by mkbuiltin.go and generates builtin.go. It should be named builtin too, not builtins, both for consistency and because file and directory names in general are singular unless forced otherwise. Commented on CL 6233 too. Change-Id: Ic5d3671443ae9292b69fda118f61a11c88d823fa Reviewed-on: https://go-review.googlesource.com/7660Reviewed-by:
Minux Ma <minux@golang.org>
-
Russ Cox authored
Also replace proginfo call with cheaper calls where only flags are needed. Change-Id: Ib6e5c12bd8752b87c0d8bcf22fa9e25e04a7941f Reviewed-on: https://go-review.googlesource.com/7630Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
- avoid copy in range ytab - add fast path to prefixof Change-Id: I88aa9d91a0abe80d253f7c3bca950b4613297499 Reviewed-on: https://go-review.googlesource.com/7628 Run-TryBot: Russ Cox <rsc@golang.org> Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
Change-Id: Iaf5a7d25e6308b32c17a38afbbd46befa17aa3a4 Reviewed-on: https://go-review.googlesource.com/7629Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
These were introduced during C -> Go translation when the loop increment contained multiple statements. Change-Id: Ic8abd8dcb3308851a1f7024de00711f0f984e684 Reviewed-on: https://go-review.googlesource.com/7627Reviewed-by:
Josh Bleecher Snyder <josharian@gmail.com> Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
Change-Id: I18f2e2ee141ebb65a8579ee1e440cb9c2069ef86 Reviewed-on: https://go-review.googlesource.com/7626Reviewed-by:
Rob Pike <r@golang.org> Reviewed-by:
Minux Ma <minux@golang.org>
-
Russ Cox authored
Substituting in multiple passes meant walking the type multiple times, and worse, if a complex type was substituted in an early pass, later passes would follow it, possibly recursively, until hitting the depth 10 limit. Change-Id: Ie61d6ec08438e297baabe932afe33d08f358e55f Reviewed-on: https://go-review.googlesource.com/7625Reviewed-by:
Josh Bleecher Snyder <josharian@gmail.com> Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
Change-Id: I9fb5c1c11a750766ae2d9532869d5ab26f1cf9cf Reviewed-on: https://go-review.googlesource.com/7624Reviewed-by:
Rob Pike <r@golang.org> Reviewed-by:
Dave Cheney <dave@cheney.net>
-
Russ Cox authored
In addition to possibly being clearer code, this replaces an O(n) lookup with an O(log n) lookup. Change-Id: I0a574c536a965a87f7ad6dcdcc30f737bc771cd5 Reviewed-on: https://go-review.googlesource.com/7623Reviewed-by:
Rob Pike <r@golang.org>
-
- 16 Mar, 2015 24 commits
-
-
Joël Stemmer authored
Commit f1d669ae added support for AES_256_GCM_SHA384 cipher suites as specified in RFC5289. However, it did not take the arbitrary hash function into account in the TLS client handshake when using client certificates. The hashForClientCertificate method always returned SHA256 as its hashing function, even if it actually used a different one to calculate its digest. Setting up the connection would eventually fail with the error "tls: failed to sign handshake with client certificate: crypto/rsa: input must be hashed message". Included is an additional test for this specific situation that uses the SHA384 hash. Fixes #9808 Change-Id: Iccbf4ab225633471ef897907c208ad31f92855a3 Reviewed-on: https://go-review.googlesource.com/7040Reviewed-by:
Adam Langley <agl@golang.org> Run-TryBot: Adam Langley <agl@golang.org>
-
Nick Sullivan authored
Decrypter is an interface to support opaque private keys that perform decryption operations. This interface is analogous to the crypto.Signer interface. This change introduces the crypto.Decrypter interface and implements the crypto.Decrypter interface for rsa.PrivateKey with both OAEP and PKCS#1 v1.5 padding modes. Change-Id: I433f649f84ed3c2148337d735cafd75f1d94a904 Reviewed-on: https://go-review.googlesource.com/3900Reviewed-by:
Adam Langley <agl@golang.org>
-
Russ Cox authored
Change-Id: Ic22e004b43bd98e712befb30684be16d8214c94a Reviewed-on: https://go-review.googlesource.com/7622Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
Change-Id: I105c1e7730c1e7ccf36297b9cbf96dc0a4868013 Reviewed-on: https://go-review.googlesource.com/7621Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
Change-Id: I91fe72c60d6c41644780474620e05380e9af2a3d Reviewed-on: https://go-review.googlesource.com/7620Reviewed-by:
Josh Bleecher Snyder <josharian@gmail.com> Reviewed-by:
Rob Pike <r@golang.org>
-
Russ Cox authored
This came up in private mail. It works today and I want to make sure it stays working. Change-Id: I13ebdc2dfadb3c72d7f179be89883137320c05d0 Reviewed-on: https://go-review.googlesource.com/7390Reviewed-by:
Austin Clements <austin@google.com> Reviewed-by:
Rob Pike <r@golang.org>
-
Rob Pike authored
An explicit nil in an expression like nil.Foo caused a panic because the evaluator attempted to reflect on the nil. A typeless nil like this cannot be used to do anything, so just error out. Fixes #9426 Change-Id: Icd2c9c7533dda742748bf161eced163991a12f54 Reviewed-on: https://go-review.googlesource.com/7643Reviewed-by:
David Symonds <dsymonds@golang.org>
-
Shenghou Ma authored
While we're here, also fix two HTML issues. Fixes #9235. Change-Id: I6e2f50931c0f387881271484a726ac2308518cf4 Reviewed-on: https://go-review.googlesource.com/7602Reviewed-by:
Rob Pike <r@golang.org>
-
Josh Bleecher Snyder authored
Change-Id: Id5ce859bd4b6318dc9104f7377ae23d7f0bc30cd Reviewed-on: https://go-review.googlesource.com/7640Reviewed-by:
Minux Ma <minux@golang.org>
-
Aram Hăvărneanu authored
ARM64 doesn't have disassembler yet. Change-Id: I016fa013b5ff50dc49d38ade42351b79be023d80 Reviewed-on: https://go-review.googlesource.com/7149Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
Change-Id: I6ca9caec8ccf12618e56dcf6b83328e7acf8b1ec Reviewed-on: https://go-review.googlesource.com/7148Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Dave Cheney <dave@cheney.net> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
Change-Id: I5d40e04395de743a8fdcfa8bdc0e580729bc66a3 Reviewed-on: https://go-review.googlesource.com/7147Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Dave Cheney <dave@cheney.net> Reviewed-by:
Russ Cox <rsc@golang.org> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Aram Hăvărneanu authored
Change-Id: I033eecff5e5838ba677378ac884bf5f29267e880 Reviewed-on: https://go-review.googlesource.com/7146Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
Change-Id: I11cd4b5e8daf3805af0eaa83b55b20da889702f4 Reviewed-on: https://go-review.googlesource.com/7145Reviewed-by:
Dave Cheney <dave@cheney.net> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
Change-Id: Ief12e1435a40dd2eaddc3f97f63be44c4dd2e050 Reviewed-on: https://go-review.googlesource.com/7144Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Dave Cheney <dave@cheney.net> Reviewed-by:
Rob Pike <r@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
Change-Id: Ia817e78d9678a365a76fea5e4dbe8f8a5aab0bac Reviewed-on: https://go-review.googlesource.com/7143Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
Change-Id: Ibda6a5bedaff57fd161d63fc04ad260931d34413 Reviewed-on: https://go-review.googlesource.com/7142Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
Change-Id: I92b4301b64054272d78dd15c16bf6ff592acad26 Reviewed-on: https://go-review.googlesource.com/7141Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Dave Cheney <dave@cheney.net> Reviewed-by:
Russ Cox <rsc@golang.org> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-
Aram Hăvărneanu authored
Change-Id: Ia6c3d5e7a32b20e3c45d9485e66b48cd02644280 Reviewed-on: https://go-review.googlesource.com/7140Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Dave Cheney <dave@cheney.net> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
No peep optimizer yet. Change-Id: Ifa5f993cd6ac5e34783c0df41faf772fbce96ae2 Reviewed-on: https://go-review.googlesource.com/7049Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
Only internal linking without cgo is supported for now. Change-Id: I91eb1572c1ccc805db62fc4c29080df98797d51a Reviewed-on: https://go-review.googlesource.com/7048Reviewed-by:
Minux Ma <minux@golang.org> Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
Pre/post-index addressing modes with writeback use .W and .P instruction suffixes, like on ARM. Complex addressing modes are not supported yet. Change-Id: I537a1c3fe5b057c0812662677d0010bc8c468ffb Reviewed-on: https://go-review.googlesource.com/7047Reviewed-by:
Rob Pike <r@golang.org>
-
Aram Hăvărneanu authored
ARM64 (ARMv8) has 32 general purpose, 64-bit integer registers (R0-R31), 32 64-bit scalar floating point registers (F0-F31), and 32 128-bit vector registers (unused, V0-V31). R31 is either the stack pointer (RSP), or the zero register (ZR), depending on the instruction. Note the distinction between the hardware stack pointer, RSP, and the virtual stack pointer SP. The (hardware) stack pointer must be 16-byte aligned at all times; the RSP register itself must be aligned, offset(RSP) only has to have natural alignment. Instructions are fixed-width, and are 32-bit wide. ARM64 supports ARMv7 too (32-bit ARM), but not in the same process. In general, there is not much in common between 32-bit ARM and ARM64, it's a new architecture. All implementations have floating point instructions. This change adds a Prog.To3 field analogous to Prog.To. It is used by exclusive load/store instructions such as STLXR which read from one register, and write to both a register and a memory address. STLXRW R1, (R0), R3 This will store the word contained in R1 to the memory address pointed by R0. R3 will be updated with the status result of the store. It is used to implement atomic operations. No other changes are made to the portable Prog and Addr structures. Change-Id: Ie839029aa5265bbad35769d9689eca11e1c48c47 Reviewed-on: https://go-review.googlesource.com/7046Reviewed-by:
Russ Cox <rsc@golang.org>
-
Aram Hăvărneanu authored
ARM64 doesn't have the old assembler. Change-Id: I9253271029440e2b7f2813d3e98a7d2e7a65bfbc Reviewed-on: https://go-review.googlesource.com/7045Reviewed-by:
Russ Cox <rsc@golang.org> Reviewed-by:
Austin Clements <austin@google.com> Reviewed-by:
Ian Lance Taylor <iant@golang.org>
-