Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(25)

Issue 10150043: code review 10150043: cmd/cgo: makes clang happy by not using __gcc_struct__ ... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 6 months ago by minux1
Modified:
12 years, 6 months ago
Reviewers:
iant
CC:
golang-dev, iant
Visibility:
Public.

Description

cmd/cgo: makes clang happy by not using __gcc_struct__ attribute.

Patch Set 1 #

Patch Set 2 : diff -r 9b1143bb425e https://code.google.com/p/go/ #

Patch Set 3 : diff -r 9b1143bb425e https://code.google.com/p/go/ #

Patch Set 4 : diff -r b08e22305bcc https://code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/cmd/cgo/out.go View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5
minux1
Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go/
12 years, 6 months ago (2013-06-10 14:26:39 UTC) #1
iant
LGTM Doesn't this mean that we are going to have the same bug with clang ...
12 years, 6 months ago (2013-06-10 17:01:28 UTC) #2
minux1
On Tue, Jun 11, 2013 at 1:01 AM, <iant@golang.org> wrote: > Doesn't this mean that ...
12 years, 6 months ago (2013-06-10 18:37:57 UTC) #3
minux1
*** Submitted as https://code.google.com/p/go/source/detail?r=578baa4acf63 *** cmd/cgo: makes clang happy by not using __gcc_struct__ attribute. R=golang-dev, ...
12 years, 6 months ago (2013-06-10 18:51:11 UTC) #4
iant
12 years, 6 months ago (2013-06-10 19:35:37 UTC) #5
On Mon, Jun 10, 2013 at 11:37 AM, minux <minux.ma@gmail.com> wrote:
>
> btw, why does the GCC PR remain unfixed for so long (there is even a patch
> on the bugzilla)?

I don't know why it hasn't been fixed.  The patch in the bugzilla
entry seems to be wrong, but I'm still surprised that the GCC Windows
maintainers haven't pushed harder on this.  Part of the problem is of
course that most GCC developers do not use Windows.

Ian
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b