Re: Go patch committed: Harmonize types referenced by both C and Go

2019-02-19 Thread Rainer Orth
Hi Ian, > On Mon, Feb 18, 2019 at 2:48 AM Rainer Orth > wrote: >> >> > The code was already calling syscall, it was just doing it in a way >> > that the types didn't necessarily match the C declaration. This is >> > the implementation of Go's syscall.Syscall function, so there isn't >> >

Re: Go patch committed: Harmonize types referenced by both C and Go

2019-02-18 Thread Ian Lance Taylor
On Mon, Feb 18, 2019 at 2:48 AM Rainer Orth wrote: > > > The code was already calling syscall, it was just doing it in a way > > that the types didn't necessarily match the C declaration. This is > > the implementation of Go's syscall.Syscall function, so there isn't > > really anything else we

Re: Go patch committed: Harmonize types referenced by both C and Go

2019-02-18 Thread Rainer Orth
Hi Ian, > On Fri, Feb 15, 2019 at 4:03 AM Rainer Orth > wrote: >> >> Andreas Schwab writes: >> >> > This breaks non-split-stack builds. >> > >> > ../../../libgo/runtime/stack.c: In function 'doscanstack1': >> > ../../../libgo/runtime/stack.c:113:18: error: passing argument 1 of >> >

Re: Go patch committed: Harmonize types referenced by both C and Go

2019-02-15 Thread Ian Lance Taylor
On Fri, Feb 15, 2019 at 4:03 AM Rainer Orth wrote: > > Andreas Schwab writes: > > > This breaks non-split-stack builds. > > > > ../../../libgo/runtime/stack.c: In function 'doscanstack1': > > ../../../libgo/runtime/stack.c:113:18: error: passing argument 1 of > > 'scanstackblock' makes integer

Re: Go patch committed: Harmonize types referenced by both C and Go

2019-02-15 Thread Ian Lance Taylor
On Fri, Feb 15, 2019 at 12:15 AM Andreas Schwab wrote: > > This breaks non-split-stack builds. > > ../../../libgo/runtime/stack.c: In function 'doscanstack1': > ../../../libgo/runtime/stack.c:113:18: error: passing argument 1 of > 'scanstackblock' makes integer from pointer without a cast >

Re: Go patch committed: Harmonize types referenced by both C and Go

2019-02-15 Thread Rainer Orth
Andreas Schwab writes: > This breaks non-split-stack builds. > > ../../../libgo/runtime/stack.c: In function 'doscanstack1': > ../../../libgo/runtime/stack.c:113:18: error: passing argument 1 of > 'scanstackblock' makes integer from pointer without a cast > [-Werror=int-conversion] > 113 |

Re: Go patch committed: Harmonize types referenced by both C and Go

2019-02-15 Thread Andreas Schwab
This breaks non-split-stack builds. ../../../libgo/runtime/stack.c: In function 'doscanstack1': ../../../libgo/runtime/stack.c:113:18: error: passing argument 1 of 'scanstackblock' makes integer from pointer without a cast [-Werror=int-conversion] 113 | scanstackblock(bottom, (uintptr)(top

Go patch committed: Harmonize types referenced by both C and Go

2019-02-14 Thread Ian Lance Taylor
This patch to the Go frontend and libgo by Nikhil Benesch harmonizes types referenced by both C and Go. Compiling with LTO revealed a number of cases in the runtime and standard library where C and Go disagreed about the type of an object or function (or where Go and code generated by the