Skip to content

G
golang
Switch branch/tag
  1. 02 Dec, 2013 3 commits
  3. 01 Dec, 2013 1 commit
  5. 27 Nov, 2013 1 commit
  7. 25 Nov, 2013 2 commits
  9. 20 Nov, 2013 2 commits
  11. 19 Nov, 2013 3 commits
  13. 18 Nov, 2013 2 commits
  15. 15 Nov, 2013 1 commit
  17. 14 Nov, 2013 2 commits
  19. 13 Nov, 2013 7 commits
  21. 12 Nov, 2013 2 commits
  23. 11 Nov, 2013 2 commits
  25. 08 Nov, 2013 2 commits
  27. 07 Nov, 2013 1 commit
    • Russ Cox's avatar
      cmd/cgo: fix handling of array of pointers when using clang · 6be1cb8c
      Russ Cox authored 
      Clang does not record the "size" field for pointer types,
so we must insert the size ourselves. We were already
doing this, but only for the case of pointer types.
For an array of pointer types, the setting of the size for
the nested pointer type was happening after the computation
of the size of the array type, meaning that the array type
was always computed as 0 bytes. Delay the size computation.

This bug happens on all Clang systems, not just FreeBSD.
Our test checked that cgo wrote something, not that it was correct.
FreeBSD's default clang rejects array[0] as a C struct field,
so it noticed the incorrect sizes. But the sizes were incorrect
everywhere.

Update testcdefs to check the output has the right semantics.

Fixes #6292.

R=golang-dev, iant
CC=golang-dev
https://golang.org/cl/22840043
      6be1cb8c
  29. 06 Nov, 2013 1 commit
  31. 05 Nov, 2013 3 commits
  33. 04 Nov, 2013 2 commits
  35. 01 Nov, 2013 2 commits
  37. 31 Oct, 2013 1 commit
    • Russ Cox's avatar
      cmd/5l, runtime: fix divide for profiling tracebacks on ARM · 2c98a3bc
      Russ Cox authored 
      Two bugs:
1. The first iteration of the traceback always uses LR when provided,
which it is (only) during a profiling signal, but in fact LR is correct
only if the stack frame has not been allocated yet. Otherwise an
intervening call may have changed LR, and the saved copy in the stack
frame should be used. Fix in traceback_arm.c.

2. The division runtime call adds 8 bytes to the stack. In order to
keep the traceback routines happy, it must copy the saved LR into
the new 0(SP). Change

        SUB $8, SP

into

        MOVW    0(SP), R11 // r11 is temporary, for use by linker
        MOVW.W  R11, -8(SP)

to update SP and 0(SP) atomically, so that the traceback always
sees a saved LR at 0(SP).

Fixes #6681.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/19910044
      2c98a3bc