1. 03 Jun, 2019 5 commits
  2. 02 Jun, 2019 2 commits
  3. 31 May, 2019 9 commits
  4. 30 May, 2019 13 commits
  5. 29 May, 2019 6 commits
    • Matthew Dempsky's avatar
      cmd/compile: fix fmt_test.go after CL 170062 · cd24849e
      Matthew Dempsky authored
      Updates #22326.
      
      Change-Id: Ia9173b6eb29b2a4f90f4ba39bf53b6e9b7a6d6bf
      Reviewed-on: https://go-review.googlesource.com/c/go/+/179398
      Run-TryBot: Matthew Dempsky <mdempsky@google.com>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: default avatarBrad Fitzpatrick <bradfitz@golang.org>
      cd24849e
    • Matthew Dempsky's avatar
      cmd/compile: sort OAS2* declarations · 913d2904
      Matthew Dempsky authored
      Change-Id: Idd3acf5f808705c608cd4e5877bc93e1626d9a58
      Reviewed-on: https://go-review.googlesource.com/c/go/+/179238
      Run-TryBot: Matthew Dempsky <mdempsky@google.com>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: default avatarRobert Griesemer <gri@golang.org>
      913d2904
    • Matthew Dempsky's avatar
      cmd/compile: fix package initialization ordering · 5d0d87ae
      Matthew Dempsky authored
      This CL rewrites cmd/compile's package-level initialization ordering
      algorithm to be compliant with the Go spec. See documentation in
      initorder.go for details.
      
      Incidentally, this CL also improves fidelity of initialization loop
      diagnostics by including referenced functions in the emitted output
      like go/types does.
      
      Fixes #22326.
      
      Change-Id: I7c9ac47ff563df4d4f700cf6195387a0f372cc7b
      Reviewed-on: https://go-review.googlesource.com/c/go/+/170062
      Run-TryBot: Matthew Dempsky <mdempsky@google.com>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: default avatarRobert Griesemer <gri@golang.org>
      5d0d87ae
    • Jason A. Donenfeld's avatar
      cmd/link: do not generate NT 4 compatibility binaries · e883d000
      Jason A. Donenfeld authored
      Incredibly, the subsystem version numbers in the PE header influence how
      win32k handles various syscalls. The first time a win32k syscall is
      invoked and the kernel upgrades the thread object to a tagTHREADINFO
      with all of the lovely undocumented UI members and such, it sets the
      dwExpWinVer member (offset 624 in Windows 10 build 1809) to the result
      of RtlGetExpWinVer(PsGetProcessSectionBaseAddress(proc)).
      RtlGetExpWinVer, also undocumented, then calls into the undocumented
      RtlImageNtHeader function, which returns a fortunately documented
      IMAGE_NT_HEADERS structure. It uses the subsystem members in there to
      set the dwExpWinVer member of our newly minted tagTHREADINFO object.
      Later, functions like SendInput consult this to vary their behaviors and
      return values. In fact, littered through out win32k are checks like `if
      (gsti->dwExpWinVer >= 0x501) { ... }`.
      
      I don't think Go ever supported NT 4.0. These days the minimum version
      is Windows 7, which is 6.1. So, let's set the version numbers in the PE
      header at that, which should give us the behavior that MSDN advertises
      for various functions, as opposed to bizarre archeological remnants.
      
      Interestingly, I suspect that most people never noticed the brokenness,
      because most people trying to do serious Win32 UI stuff wind up linking
      in cgo, if not for actually using C, then just to have a larger system
      stack so that the stack doesn't get corrupted by various UI functions.
      When MingW is used, the PE header gets a later version. But recently
      there's been a bug report of some people trying to do more modest UI
      manipulation using SendInput in a setting where this cgo hack probably
      isn't required, so they ran into the weird historical compatibility
      stuff.
      
      Fixes #31685
      
      Change-Id: I54461ce820f6e9df349e37be5ecc5a44c04a3e26
      Reviewed-on: https://go-review.googlesource.com/c/go/+/178977
      Run-TryBot: Jason Donenfeld <Jason@zx2c4.com>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: default avatarAlex Brainman <alex.brainman@gmail.com>
      Reviewed-by: default avatarBrad Fitzpatrick <bradfitz@golang.org>
      e883d000
    • Russ Cox's avatar
      cmd/go: update docs for GOPROXY default · e12efecf
      Russ Cox authored
      The default is now "https://proxy.golang.org,direct" not just "https://proxy.golang.org"
      (as of CL 178720).
      
      Change-Id: I47e67602e994332b988ed1b75fa57319f3876cc6
      Reviewed-on: https://go-review.googlesource.com/c/go/+/178725
      Run-TryBot: Russ Cox <rsc@golang.org>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: default avatarBryan C. Mills <bcmills@google.com>
      e12efecf
    • Keith Randall's avatar
      cmd/compile: add GCNode to VarDef in tests · 200176ed
      Keith Randall authored
      Change-Id: I0f79d317ae7ac46c7e824bd382a4f51d53adaaa6
      Reviewed-on: https://go-review.googlesource.com/c/go/+/179257
      Run-TryBot: Keith Randall <khr@golang.org>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: default avatarJosh Bleecher Snyder <josharian@gmail.com>
      200176ed
  6. 28 May, 2019 5 commits
    • Brad Fitzpatrick's avatar
      src/vendor: update golang.org/x/sys to remove x/sys/unix dep · fa02af42
      Brad Fitzpatrick authored
      Fixes #32102
      
      Change-Id: I3022104035a12bf31a7cd214aede58c024effc1b
      Reviewed-on: https://go-review.googlesource.com/c/go/+/179180Reviewed-by: default avatarBryan C. Mills <bcmills@google.com>
      fa02af42
    • Brad Fitzpatrick's avatar
      net/http: quiet some log spam in tests · 770746af
      Brad Fitzpatrick authored
      One of these tests creates a bunch of connections concurrently, then
      discovers it doesn't need them all, which then makes the server log
      that the client went away midway through the TLS handshake. Perhaps
      the server should recognize that as a case not worthy of logging
      about, but this is a safer way to eliminate the stderr spam during go
      test for now.
      
      The other test's client gives up on its connection and closes it,
      similarly confusing the server.
      
      Change-Id: I49ce442c9a63fc437e58ca79f044aa76e8c317b5
      Reviewed-on: https://go-review.googlesource.com/c/go/+/179179
      Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: default avatarDmitri Shuralyov <dmitshur@golang.org>
      770746af
    • Josh Bleecher Snyder's avatar
      cmd/compile: process blocks containing only dead values in fuseIf · ddb5d0db
      Josh Bleecher Snyder authored
      The code in #29218 resulted in an If block containing only its control.
      That block was then converted by fuseIf into a plain block;
      as a result, that control value was dead.
      However, the control value was still present in b.Values.
      This prevented further fusing of that block.
      
      This change beefs up the check in fuseIf to allow fusing
      blocks that contain only dead values (if any).
      In the case of #29218, this enables enough extra
      fusing that the control value could be eliminated,
      allowing all values in turn to be eliminated.
      
      This change also fuses 34 new blocks during make.bash.
      
      It is not clear that this fixes every variant of #29218,
      but it is a reasonable standalone change.
      And code like #29218 is rare and fundamentally buggy,
      so we can handle new instances if/when they actually occur.
      
      Fixes #29218
      
      Negligible toolspeed impact.
      
      name        old time/op       new time/op       delta
      Template          213ms ± 3%        213ms ± 2%    ~     (p=0.914 n=97+88)
      Unicode          89.8ms ± 2%       89.6ms ± 2%  -0.22%  (p=0.045 n=93+95)
      GoTypes           712ms ± 3%        709ms ± 2%  -0.35%  (p=0.023 n=95+95)
      Compiler          3.24s ± 2%        3.23s ± 2%  -0.30%  (p=0.020 n=98+97)
      SSA               10.0s ± 1%        10.0s ± 1%    ~     (p=0.382 n=98+99)
      Flate             135ms ± 3%        135ms ± 2%    ~     (p=0.983 n=98+98)
      GoParser          158ms ± 2%        158ms ± 2%    ~     (p=0.170 n=99+99)
      Reflect           447ms ± 3%        447ms ± 2%    ~     (p=0.538 n=98+89)
      Tar               189ms ± 2%        189ms ± 3%    ~     (p=0.874 n=95+96)
      XML               251ms ± 2%        251ms ± 2%    ~     (p=0.434 n=94+96)
      [Geo mean]        427ms             426ms       -0.15%
      
      name        old user-time/op  new user-time/op  delta
      Template          264ms ± 2%        265ms ± 2%    ~     (p=0.075 n=96+90)
      Unicode           119ms ± 6%        119ms ± 7%    ~     (p=0.864 n=99+98)
      GoTypes           926ms ± 2%        924ms ± 2%    ~     (p=0.071 n=94+94)
      Compiler          4.38s ± 2%        4.37s ± 2%  -0.34%  (p=0.001 n=98+97)
      SSA               13.4s ± 1%        13.4s ± 1%    ~     (p=0.693 n=90+93)
      Flate             162ms ± 3%        161ms ± 2%    ~     (p=0.163 n=99+99)
      GoParser          186ms ± 2%        186ms ± 3%    ~     (p=0.130 n=96+100)
      Reflect           572ms ± 3%        572ms ± 2%    ~     (p=0.608 n=97+97)
      Tar               239ms ± 2%        239ms ± 3%    ~     (p=0.999 n=93+91)
      XML               302ms ± 2%        302ms ± 2%    ~     (p=0.627 n=91+97)
      [Geo mean]        540ms             540ms       -0.08%
      
      file    before    after     Δ       %       
      asm     4862704   4858608   -4096   -0.084% 
      compile 24001568  24001680  +112    +0.000% 
      total   132520780 132516796 -3984   -0.003% 
      
      file                       before    after     Δ       %       
      cmd/compile/internal/gc.a  8887638   8887596   -42     -0.000% 
      cmd/compile/internal/ssa.a 29995056  29998986  +3930   +0.013% 
      cmd/internal/obj/wasm.a    209444    203652    -5792   -2.765% 
      total                      129471798 129469894 -1904   -0.001% 
      
      Change-Id: I2d18f9278e68b9766058ae8ca621e844f9d89dd8
      Reviewed-on: https://go-review.googlesource.com/c/go/+/177140
      Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      Reviewed-by: default avatarKeith Randall <khr@golang.org>
      Reviewed-by: default avatarCuong Manh Le <cuong.manhle.vn@gmail.com>
      ddb5d0db
    • Dmitri Shuralyov's avatar
      testing/internal/testdeps: delete didSetLogger variable · bd61fc3d
      Dmitri Shuralyov authored
      It was added in CL 83956 but never used.
      
      Updates #23129
      
      Change-Id: I70b50e974a56620069a77658386722af314cc857
      Reviewed-on: https://go-review.googlesource.com/c/go/+/179138
      Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
      Reviewed-by: default avatarBrad Fitzpatrick <bradfitz@golang.org>
      TryBot-Result: Gobot Gobot <gobot@golang.org>
      bd61fc3d
    • Maxim Eryomenko's avatar
      bytes: remove obsolete comment · 0f897f91
      Maxim Eryomenko authored
      Change-Id: Iec4348eb674c8c64497c82e5d7cd656736cfcebd
      GitHub-Last-Rev: 462c323960be0ca1c126c051ab97f72949bdc2b6
      GitHub-Pull-Request: golang/go#32225
      Reviewed-on: https://go-review.googlesource.com/c/go/+/178797Reviewed-by: default avatarBrad Fitzpatrick <bradfitz@golang.org>
      0f897f91