1. 13 Sep, 2012 1 commit
    • Andrew Gerrand's avatar
      undo CL 6493071 / 56c7453d488f · e7d7ea21
      Andrew Gerrand authored
      This change messes with anchor links. It obscures the item being linked to.
      I don't see a way around it. Undoing for now.
      
            Fixes #4071.
      
            ««« original CL description
            doc css: topbar sticks to the top of large windows.
            Rationale: for large screens, the convenience of not having to scroll
            to the top of the page to do a search outweighs having less vertical
            space.
            Tested with Chrome, Firefox, Safari with various window and text sizes.
      
            R=adg
            CC=gobot, golang-dev
            https://golang.org/cl/6493071
      
            Committer: Andrew Gerrand <adg@golang.org>
            »»»
      
      R=golang-dev, r, dsymonds
      CC=golang-dev
      https://golang.org/cl/6488121
      e7d7ea21
  2. 12 Sep, 2012 10 commits
  3. 11 Sep, 2012 11 commits
    • Daniel Morsing's avatar
      cmd/gc: Inline pointer sized T2I interface conversions · 8fd65b0e
      Daniel Morsing authored
      This CL also adds support for marking the likelyness of IF nodes in the AST being true. This feature is being used here to mark the slow path as unlikely.
      
      src/pkg/runtime:
      benchmark                  old ns/op    new ns/op    delta
      BenchmarkConvT2IUintptr           16            1  -91.63%
      
      test/bench/go1:
      benchmark                 old ns/op    new ns/op    delta
      BenchmarkBinaryTree17    5416917000   5461355000   +0.82%
      BenchmarkFannkuch11      3810355000   3842609000   +0.85%
      BenchmarkGobDecode         19950950     19855420   -0.48%
      BenchmarkGobEncode         11301220     11308530   +0.06%
      BenchmarkGzip             548119600    546869200   -0.23%
      BenchmarkGunzip           176145400    180208300   +2.31%
      BenchmarkJSONEncode        93117400     70163100  -24.65%
      BenchmarkJSONDecode       406626800    409999200   +0.83%
      BenchmarkMandelbrot200      6300992      6317866   +0.27%
      BenchmarkParse              7664396      7451625   -2.78%
      BenchmarkRevcomp         1189424000   1412332000  +18.74%
      BenchmarkTemplate         491308400    458654200   -6.65%
      
      benchmark                  old MB/s     new MB/s  speedup
      BenchmarkGobDecode            38.47        38.66    1.00x
      BenchmarkGobEncode            67.92        67.87    1.00x
      BenchmarkGzip                 35.40        35.48    1.00x
      BenchmarkGunzip              110.16       107.68    0.98x
      BenchmarkJSONEncode           20.84        27.66    1.33x
      BenchmarkJSONDecode            4.77         4.73    0.99x
      BenchmarkParse                 7.56         7.77    1.03x
      BenchmarkRevcomp             213.69       179.96    0.84x
      BenchmarkTemplate              3.95         4.23    1.07x
      
      R=rsc, dave, nigeltao
      CC=golang-dev
      https://golang.org/cl/6351090
      8fd65b0e
    • Nigel Tao's avatar
      cmd/gc: recognize small TPTR64 values as small integer constants. · 0184081e
      Nigel Tao authored
      Given the following Go program:
      
      func sum(s []int) int {
              ret := 0
              for _, x := range s {
                      ret += x
              }
              return ret
      }
      
      6g would previously generate:
      
      --- prog list "sum" ---
      0000 (main.go:3) TEXT    sum+0(SB),$0-24
      0001 (main.go:5) MOVQ    s+0(FP),CX
      0002 (main.go:5) MOVL    s+8(FP),DI
      0003 (main.go:5) MOVL    s+12(FP),BX
      0004 (main.go:4) MOVL    $0,SI
      0005 (main.go:5) MOVL    $0,AX
      0006 (main.go:5) JMP     ,8
      0007 (main.go:5) INCL    ,AX
      0008 (main.go:5) CMPL    AX,DI
      0009 (main.go:5) JGE     $0,16
      0010 (main.go:5) MOVL    (CX),DX
      0011 (main.go:5) MOVQ    $4,BX
      0012 (main.go:5) ADDQ    CX,BX
      0013 (main.go:5) MOVQ    BX,CX
      0014 (main.go:6) ADDL    DX,SI
      0015 (main.go:5) JMP     ,7
      0016 (main.go:8) MOVL    SI,.noname+16(FP)
      0017 (main.go:8) RET     ,
      
      and now generates:
      
      --- prog list "sum" ---
      0000 (main.go:3) TEXT    sum+0(SB),$0-24
      0001 (main.go:5) MOVQ    s+0(FP),CX
      0002 (main.go:5) MOVL    s+8(FP),DI
      0003 (main.go:5) MOVL    s+12(FP),BX
      0004 (main.go:4) MOVL    $0,SI
      0005 (main.go:5) MOVL    $0,AX
      0006 (main.go:5) JMP     ,8
      0007 (main.go:5) INCL    ,AX
      0008 (main.go:5) CMPL    AX,DI
      0009 (main.go:5) JGE     $0,14
      0010 (main.go:5) MOVL    (CX),BP
      0011 (main.go:5) ADDQ    $4,CX
      0012 (main.go:6) ADDL    BP,SI
      0013 (main.go:5) JMP     ,7
      0014 (main.go:8) MOVL    SI,.noname+16(FP)
      0015 (main.go:8) RET     ,
      
      The key difference is that
      0011 (main.go:5) MOVQ    $4,BX
      0012 (main.go:5) ADDQ    CX,BX
      0013 (main.go:5) MOVQ    BX,CX
      has changed to
      0011 (main.go:5) ADDQ    $4,CX
      
      R=rsc, dave, remyoudompheng
      CC=golang-dev
      https://golang.org/cl/6506089
      0184081e
    • Andrew Gerrand's avatar
      C: add Francesc Campoy (Google CLA) · b64a99da
      Andrew Gerrand authored
      R=golang-dev, dsymonds, campoy
      CC=golang-dev
      https://golang.org/cl/6490102
      b64a99da
    • Rémy Oudompheng's avatar
      cmd/6g, cmd/8g: do not LEA[LQ] interfaces when calling methods. · b45b6fd1
      Rémy Oudompheng authored
      It is enough to load directly the data word and the itab word
      from memory, so we save a LEA instruction for each method call,
      and allow elimination of some extra temporaries.
      
      Update #1914.
      
      R=daniel.morsing, rsc
      CC=golang-dev, remy
      https://golang.org/cl/6501110
      b45b6fd1
    • Rémy Oudompheng's avatar
      cmd/6g, cmd/8g: eliminate extra agen for nil comparisons. · ff642e29
      Rémy Oudompheng authored
      Removes an extra LEAL/LEAQ instructions there and usually saves
      a useless temporary in the idiom
          if err := foo(); err != nil {...}
      
      Generated code is also less involved:
          MOVQ err+n(SP), AX
          CMPQ AX, $0
      (potentially CMPQ n(SP), $0) instead of
          LEAQ err+n(SP), AX
          CMPQ (AX), $0
      
      Update #1914.
      
      R=daniel.morsing, nigeltao, rsc
      CC=golang-dev, remy
      https://golang.org/cl/6493099
      ff642e29
    • Nigel Tao's avatar
      strings: more thorough Replacer tests. · b19c32ac
      Nigel Tao authored
      This verifies existing behavior. Some replacements are arguably wrong
      (these are marked with TODO) but changing behavior is left for a
      follow-up CL.
      
      Also fix that BenchmarkGenericMatch wasn't actually matching anything.
      
      R=rsc, eric.d.eisner
      CC=bradfitz, golang-dev
      https://golang.org/cl/6488110
      b19c32ac
    • Andrew Gerrand's avatar
      C: spell Brian's name correctly · b4840366
      Andrew Gerrand authored
      R=iant, nigeltao
      CC=golang-dev
      https://golang.org/cl/6492102
      b4840366
    • Alex Brainman's avatar
      4c60f974
    • Russ Cox's avatar
      encoding/json: handle anonymous fields · f97bfb93
      Russ Cox authored
      Fixes #3069.
      
      R=golang-dev, n13m3y3r
      CC=golang-dev
      https://golang.org/cl/6460044
      f97bfb93
    • Brian Slesinskya's avatar
      doc css: topbar sticks to the top of large windows. · 56e1384a
      Brian Slesinskya authored
      Rationale: for large screens, the convenience of not having to scroll
      to the top of the page to do a search outweighs having less vertical
      space.
      Tested with Chrome, Firefox, Safari with various window and text sizes.
      
      R=adg
      CC=gobot, golang-dev
      https://golang.org/cl/6493071
      56e1384a
    • Andrew Gerrand's avatar
      C: add Brian Slesinsky (Google CLA) · 47ae8a28
      Andrew Gerrand authored
      R=golang-dev, dsymonds
      CC=golang-dev
      https://golang.org/cl/6496103
      47ae8a28
  4. 10 Sep, 2012 11 commits
  5. 09 Sep, 2012 2 commits
    • Rémy Oudompheng's avatar
      cmd/8g: import componentgen from 6g. · ae0862c1
      Rémy Oudompheng authored
      This makes the compilers code more similar and improves
      code generation a lot.
      
      The number of LEAL instructions generated for cmd/go drops
      by 60%.
      
      % GOARCH=386 go build -gcflags -S -a cmd/go | grep LEAL | wc -l
      Before:       89774
      After:        47548
      
      benchmark                              old ns/op    new ns/op    delta
      BenchmarkAppendFloatDecimal                  540          444  -17.78%
      BenchmarkAppendFloat                        1160         1035  -10.78%
      BenchmarkAppendFloatExp                     1060          922  -13.02%
      BenchmarkAppendFloatNegExp                  1053          920  -12.63%
      BenchmarkAppendFloatBig                     1773         1558  -12.13%
      BenchmarkFormatInt                         13065        12481   -4.47%
      BenchmarkAppendInt                         10981         9900   -9.84%
      BenchmarkFormatUint                         3804         3650   -4.05%
      BenchmarkAppendUint                         3506         3303   -5.79%
      BenchmarkUnquoteEasy                         714          683   -4.34%
      BenchmarkUnquoteHard                        5117         2915  -43.03%
      
      Update #1914.
      
      R=nigeltao, rsc, golang-dev
      CC=golang-dev, remy
      https://golang.org/cl/6489067
      ae0862c1
    • Mikio Hara's avatar
      net: change cname lookup test target to iana.org · b7376442
      Mikio Hara authored
      It might be meaningful, and stable than others.
      
      R=golang-dev, agl
      CC=golang-dev
      https://golang.org/cl/6499091
      b7376442
  6. 08 Sep, 2012 3 commits
  7. 07 Sep, 2012 2 commits