[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-11-21 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 Eric Botcazou changed: What|Removed |Added Status|WAITING |SUSPENDED --- Comment #60 from Eric Botca

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-11-21 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 Iain Sandoe changed: What|Removed |Added Status|NEW |WAITING --- Comment #59 from Iain Sandoe 2

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-11-21 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #58 from Iain Sandoe 2011-11-21 09:04:14 UTC --- Author: iains Date: Mon Nov 21 09:04:08 2011 New Revision: 181553 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181553 Log: gcc/ada: Backport from mainline r181474 PR

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-11-18 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #57 from Iain Sandoe 2011-11-18 13:19:29 UTC --- Author: iains Date: Fri Nov 18 13:19:25 2011 New Revision: 181474 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181474 Log: gcc/ada: PR target/50678 * init.c (__gnat_

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-28 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #56 from Iain Sandoe 2011-10-28 12:01:10 UTC --- please leave this bug open for now - it is not really fixed, the patch applied is a workaround. once we get a response to the radar, we'll know better how to proceed.

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-28 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #55 from Iain Sandoe 2011-10-28 11:59:10 UTC --- Author: iains Date: Fri Oct 28 11:59:07 2011 New Revision: 180613 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=180613 Log: ada: PR target/50678 * init.c (Darwin/__gn

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-26 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #54 from Eric Botcazou 2011-10-26 20:15:43 UTC --- > well, no reply to my radar yet - > - so, shall I apply the proposed patch as "sticking plaster" until we decide > the Right Way forward ? OK, let's go ahead.

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-26 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #53 from Iain Sandoe 2011-10-26 19:57:25 UTC --- well, no reply to my radar yet - - so, shall I apply the proposed patch as "sticking plaster" until we decide the Right Way forward ? (I believe it's generally been approved, just a qu

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-26 Thread simon at pushface dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 simon at pushface dot org changed: What|Removed |Added CC||simon at pushface dot org ---

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-18 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #51 from Eric Botcazou 2011-10-18 20:04:38 UTC --- > 2. if the vendor decides to 'fix' libunwind .. we won't detect this ... > (although I still think this idea is worth pursuing, on the grounds that 'no > fix' or a fix to Libc are m

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-18 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #50 from Iain Sandoe 2011-10-18 19:18:38 UTC --- well I've hit a few issues. 1. _Unwind_Find_FDE is not part of the public interface (nor are the types it needs). 2. if the vendor decides to 'fix' libunwind .. we won't detect this ..

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-18 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #49 from Iain Sandoe 2011-10-18 17:26:28 UTC --- (In reply to comment #48) > I have one question: how this unwinder problem relates to the other ones > reported in bugzilla? To the bugs I am aware of, totally unrelated ... This pro

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-18 Thread dominiq at lps dot ens.fr
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #48 from Dominique d'Humieres 2011-10-18 17:06:44 UTC --- I have bootstrapped gcc on x86_64-apple-darwin10 at revision 180138 with the patch at http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01617.html. All the Ada tests pass with it. T

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-18 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #47 from Iain Sandoe 2011-10-18 16:22:10 UTC --- (In reply to comment #46) > > that seems reasonable if the result can be cached - otherwise it's > > potentially > > a big hit. > > We don't really care about performances here: a sig

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-18 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #46 from Eric Botcazou 2011-10-18 16:03:20 UTC --- > that seems reasonable if the result can be cached - otherwise it's potentially > a big hit. We don't really care about performances here: a signal has been raised and we're about t

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-18 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #45 from Iain Sandoe 2011-10-18 15:32:33 UTC --- (In reply to comment #44) > > I think we'll need to apply the patch in the short/medium term and then > > figure > > out how to control it - which will depend on which system(s) a fix

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-18 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #44 from Eric Botcazou 2011-10-18 15:22:14 UTC --- > Created attachment 25540 [details] > demonstration of the fault using c++/vendor's tools > > after Eric solved my inverted-logic thinko ... > .. I reproduced using g++-4.2 > bug f

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-18 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 Iain Sandoe changed: What|Removed |Added Attachment #25520|0 |1 is obsolete|

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #42 from Eric Botcazou 2011-10-17 22:58:24 UTC --- > > Because %rbx is saved in the prologue of do_fail. > > hm. Isn't that the correct action? Yes, this is correct, but this will also restore the correct %rbx in the caller. > als

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #41 from Iain Sandoe 2011-10-17 22:52:28 UTC --- (In reply to comment #39) I'll try and cook up a radar... (against sigtramp unwind data in Libc taking into account your following comment). I guess we should not expect a fix for Da

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #40 from Eric Botcazou 2011-10-17 22:44:06 UTC --- > is there a definitive "right" or "wrong' - should I be patching > gcc/config/i386/darwin.h to match the sigtramp and bug-file against > libunwind? > or just file a bug agains the

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #39 from Eric Botcazou 2011-10-17 20:37:01 UTC --- > note that (despite the consistency within the code here) this differs from the > order in gcc/config/i386/i386.h... (although these are not call-saved, still > ... ) Yes, but this

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #38 from Iain Sandoe 2011-10-17 18:38:49 UTC --- (In reply to comment #37) > .. this doesn't fix the problem on Darwin9/m64 (with either the system > libgcc_s erm *oops* shoulda reinstalled the RTS.

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #37 from Iain Sandoe 2011-10-17 18:28:47 UTC --- .. this doesn't fix the problem on Darwin9/m64 (with either the system libgcc_s or the one from current gcc/4.7). I suppose it's failing to unwind through the sigtramp - that might req

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #36 from Iain Sandoe 2011-10-17 18:07:09 UTC --- (In reply to comment #35) > In any case, the problem is elsewhere, namely in the > unwind info for the _sigtramp function of the libc: apropos the i386 variant: > loc_expr_for_re

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #35 from Eric Botcazou 2011-10-17 15:36:11 UTC --- > 1. the code for the D10 libSystem unwind library is available from here: > http://www.opensource.apple.com/tarballs/libunwind/ Thanks for the pointer. > 2. Looking at a build of t

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 Iain Sandoe changed: What|Removed |Added Attachment #25518|0 |1 is obsolete|

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-17 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #33 from Iain Sandoe 2011-10-17 09:58:53 UTC --- Created attachment 25518 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25518 test of a foreign except thrown from a sig handler in c++ 1. the code for the D10 libSystem unwind li

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-15 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #32 from Iain Sandoe 2011-10-15 22:37:41 UTC --- thanks Eric, you're going much quicker then me ;) ... had some other things to do. I think libUnwind is released as OS these days, so it should be possible to look ... will try to find

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-15 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #31 from Eric Botcazou 2011-10-15 21:34:32 UTC --- There is some suspicious code in #0 0x7fff85c75d48 in libunwind::DwarfInstructions::stepWithDwarf(libunwind::LocalAddressSpace&, unsigned long long, unsigned long long, libunwin

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-15 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #30 from Eric Botcazou 2011-10-15 20:49:03 UTC --- > however I've not got far through Raise_From_Signal_Handler () - if one > continues from there it ends with a loop on x86-64/darwin9 and another segv on > x86-64/darwin10. You need

[Bug target/50678] [4.7 Regression] FAIL: c52104y on x86_64-apple-darwin10

2011-10-15 Thread iains at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50678 --- Comment #29 from Iain Sandoe 2011-10-15 19:33:25 UTC --- (In reply to comment #28) > > OK. well libgcc_s or libSystem contains the unwinder, depending on whether > > it's darwin9 or darwin10 (and assuming that there's no insertion caused by