Re: libgo patch committed: Update to current version of Go library

2013-02-08 Thread Ian Lance Taylor
On Fri, Feb 8, 2013 at 9:42 AM, Uros Bizjak wrote: > > Please note, that looking at generated libcalls.go, there are many > other possible signed/unsigned issues of the same type. Since this all > happens on x86_64 (where go int is apparently 64bit), it looks that it > is very important to disting

Re: libgo patch committed: Update to current version of Go library

2013-02-08 Thread Uros Bizjak
On Fri, Feb 8, 2013 at 6:10 PM, Uros Bizjak wrote: > On Fri, Feb 8, 2013 at 5:53 PM, Ian Lance Taylor wrote: >> On Fri, Feb 8, 2013 at 8:48 AM, Uros Bizjak wrote: >>> >>> utimensat returns zero on CentOS. Following is gdb session: >> >> Thanks--can you do the same session but with "step" instead

Re: libgo patch committed: Update to current version of Go library

2013-02-08 Thread Uros Bizjak
On Fri, Feb 8, 2013 at 5:53 PM, Ian Lance Taylor wrote: > On Fri, Feb 8, 2013 at 8:48 AM, Uros Bizjak wrote: >> >> utimensat returns zero on CentOS. Following is gdb session: > > Thanks--can you do the same session but with "step" instead of "next"? I think I got the problem. breakpoint at utim

Re: libgo patch committed: Update to current version of Go library

2013-02-08 Thread Ian Lance Taylor
On Fri, Feb 8, 2013 at 8:48 AM, Uros Bizjak wrote: > > utimensat returns zero on CentOS. Following is gdb session: Thanks--can you do the same session but with "step" instead of "next"? Ian

Re: libgo patch committed: Update to current version of Go library

2013-02-08 Thread Uros Bizjak
On Fri, Feb 8, 2013 at 5:21 PM, Ian Lance Taylor wrote: > On Fri, Feb 8, 2013 at 7:23 AM, Uros Bizjak wrote: >> On Fri, Feb 8, 2013 at 4:02 PM, Ian Lance Taylor wrote: >> >> I did hit one new error that seems related: >> >> --- FAIL: TestChtimes (0.00 seconds) >> os_test.go:681:

Re: libgo patch committed: Update to current version of Go library

2013-02-08 Thread Ian Lance Taylor
On Fri, Feb 8, 2013 at 7:23 AM, Uros Bizjak wrote: > On Fri, Feb 8, 2013 at 4:02 PM, Ian Lance Taylor wrote: > > I did hit one new error that seems related: > > --- FAIL: TestChtimes (0.00 seconds) > os_test.go:681: AccessTime didn't go backwards; > was={63495872497 0

Re: libgo patch committed: Update to current version of Go library

2013-02-08 Thread Uros Bizjak
On Fri, Feb 8, 2013 at 4:02 PM, Ian Lance Taylor wrote: I did hit one new error that seems related: --- FAIL: TestChtimes (0.00 seconds) os_test.go:681: AccessTime didn't go backwards; was={63495872497 0 47130825733376}, after={63495872497 0 47130825733376} >

Re: libgo patch committed: Update to current version of Go library

2013-02-08 Thread Ian Lance Taylor
On Fri, Feb 8, 2013 at 6:42 AM, Uros Bizjak wrote: > On Fri, Feb 8, 2013 at 3:07 AM, Ian Lance Taylor wrote: >> On Thu, Feb 7, 2013 at 2:43 PM, Uros Bizjak wrote: >>> >>> I did hit one new error that seems related: >>> >>> --- FAIL: TestChtimes (0.00 seconds) >>> os_test.go:681: AccessTi

Re: libgo patch committed: Update to current version of Go library

2013-02-08 Thread Uros Bizjak
On Fri, Feb 8, 2013 at 3:07 AM, Ian Lance Taylor wrote: > On Thu, Feb 7, 2013 at 2:43 PM, Uros Bizjak wrote: >> >> I did hit one new error that seems related: >> >> --- FAIL: TestChtimes (0.00 seconds) >> os_test.go:681: AccessTime didn't go backwards; >> was={63495872497 0 47130825733376

Re: libgo patch committed: Update to current version of Go library

2013-02-07 Thread Ian Lance Taylor
On Thu, Feb 7, 2013 at 2:43 PM, Uros Bizjak wrote: > > I did hit one new error that seems related: > > --- FAIL: TestChtimes (0.00 seconds) > os_test.go:681: AccessTime didn't go backwards; > was={63495872497 0 47130825733376}, after={63495872497 0 > 47130825733376} > os_test.go:685:

Re: libgo patch committed: Update to current version of Go library

2013-02-07 Thread Uros Bizjak
On Thu, Feb 7, 2013 at 5:27 PM, Ian Lance Taylor wrote: > On Thu, Feb 7, 2013 at 2:26 AM, Uros Bizjak wrote: >> >> Thanks, I have tested it on CentOS 5.9. However, for some reason, I >> got unrelated linker errors that prevents further testing: >> >> === libgo tests === >> >> ./a.

Re: libgo patch committed: Update to current version of Go library

2013-02-07 Thread Ian Lance Taylor
On Thu, Feb 7, 2013 at 2:26 AM, Uros Bizjak wrote: > > Thanks, I have tested it on CentOS 5.9. However, for some reason, I > got unrelated linker errors that prevents further testing: > > === libgo tests === > > ./a.out: error while loading shared libraries: > /home/uros/gcc-build/

Re: libgo patch committed: Update to current version of Go library

2013-02-07 Thread Uros Bizjak
On Tue, Feb 5, 2013 at 9:00 PM, Ian Lance Taylor wrote: > On Thu, Jan 3, 2013 at 1:26 AM, Uros Bizjak wrote: >> Hello! >> >>> I've committed a patch to upgrade libgo to the current version of the >>> master Go library. As usual, this mail message only includes the >>> changes to files that are s

Re: libgo patch committed: Update to current version of Go library

2013-02-05 Thread Ian Lance Taylor
On Thu, Jan 3, 2013 at 1:26 AM, Uros Bizjak wrote: > Hello! > >> I've committed a patch to upgrade libgo to the current version of the >> master Go library. As usual, this mail message only includes the >> changes to files that are specific to gccgo. Bootstrapped and ran Go >> testsuite on x86_6

Re: libgo patch committed: Update to current version of Go library

2013-01-03 Thread Uros Bizjak
Hello! > I've committed a patch to upgrade libgo to the current version of the > master Go library. As usual, this mail message only includes the > changes to files that are specific to gccgo. Bootstrapped and ran Go > testsuite on x86_64-unknown-linux-gnu. Committed to mainline. This caused m