[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-08-06 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 Uroš Bizjak changed: What|Removed |Added Target Milestone|4.10.0 |4.9.2

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-08-06 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 Uroš Bizjak changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-08-06 Thread ian at airs dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #12 from Ian Lance Taylor --- Thanks for the analysis. See also PR 60406. Dominik sent me a patch for 60406 but 1) he has no copyright assignment; 2) I think that his patch does not work for SJLJ exceptions.

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-08-06 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #11 from Uroš Bizjak --- (In reply to Andreas Schwab from comment #10) > If you never use goto *exp in the same function the value of &&label is > undefined. I did try adding goto bla: just before label, but never managed to get some

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-08-06 Thread sch...@linux-m68k.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #10 from Andreas Schwab --- If you never use goto *exp in the same function the value of &&label is undefined.

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-08-06 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #9 from Uroš Bizjak --- To illustrate unreliable approach, please compile following test: --cut here-- extern void foo (void *); int test(void) { __label__ bla; foo (&&bla); bla: return 0; } --cut here-- gcc -O2: test: .L2

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-08-06 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #8 from Uroš Bizjak --- After lots of debugging... The problem is with the label that is passed as an argument to __go_set_defer_retaddr. In function main.$thunk0, in _.179r.cse1 dump, we have: ... (insn 7 3 8 2 (set (reg:DI 84)

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-08-05 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #7 from Uroš Bizjak --- (In reply to Ian Lance Taylor from comment #6) > I don't know why making the types smaller in comment #4 makes any > difference. On a system that does not use split stacks, the runtime will > allocate a 2M sta

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-08-05 Thread ian at airs dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #6 from Ian Lance Taylor --- I don't know why making the types smaller in comment #4 makes any difference. On a system that does not use split stacks, the runtime will allocate a 2M stack for each goroutine (see StackMin in libgo/run

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-07-21 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #5 from Uroš Bizjak --- I forgot to tell that reflect test from libgo testsuite now passes: PASS: reflect

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-07-21 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #4 from Uroš Bizjak --- (In reply to Uroš Bizjak from comment #3) > (In reply to Ian Lance Taylor from comment #2) > > The patch is committed so this bug may be fixed. I haven't tested it on > > Alpha, though. > > The testcase recov

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-07-21 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 Uroš Bizjak changed: What|Removed |Added Status|WAITING |NEW --- Comment #3 from Uroš Bizjak --- (

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-07-20 Thread ian at airs dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 Ian Lance Taylor changed: What|Removed |Added Status|UNCONFIRMED |WAITING Last reconfirmed|

[Bug go/60874] FAIL: go.test/test/recover.go execution

2014-07-20 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60874 --- Comment #1 from Uroš Bizjak --- Patch that extends libgo to use libffi closures is at [1]. [1] https://gcc.gnu.org/ml/gcc-patches/2014-07/msg01355.html