http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #52 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-13
16:31:36 UTC ---
Even those this problem appears latent on gcc-4_5-branch, the problematic code
is identical. Shouldn't we backport this for gcc 4.5.3?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
m...@gcc.gnu.org mrs at gcc dot gnu.org changed:
What|Removed |Added
Known to work||4.6.0
---
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #54 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-13
18:56:48 UTC ---
(In reply to comment #53)
I'd rather know two things, that we have a lot of time on the new code and it
works fine znd why the branch doesn't have the
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #55 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-13
20:29:49 UTC ---
It appears that is was present at r162030 and the r162035 used
--enable-build-with-cxx...
http://gcc.gnu.org/ml/gcc-testresults/2010-07/msg00872.html
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #56 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-13
23:29:01 UTC ---
A regression hunt reveals the PR47324 was actually triggered by...
r160124 | hubicka | 2010-06-01 17:55:49 -0400 (Tue, 01 Jun 2010) | 4 lines
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #50 from Iain Sandoe iains at gcc dot gnu.org 2011-02-12 16:52:01
UTC ---
Author: iains
Date: Sat Feb 12 16:51:58 2011
New Revision: 170077
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=170077
Log:
gcc:
PR target/47324
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
m...@gcc.gnu.org mrs at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |RESOLVED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #48 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-11
10:09:15 UTC ---
Also eliminates PR47324 on x86_64-apple-darwin10 without additional regressions
for --enable-build-with-cxx BOOT_CFLAGS=-g -O3 CFLAGS_FOR_TARGET=-g -O3
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #49 from Iain Sandoe iains at gcc dot gnu.org 2011-02-11 10:13:13
UTC ---
(In reply to comment #48)
Also eliminates PR47324 on x86_64-apple-darwin10 without additional
regressions
for --enable-build-with-cxx BOOT_CFLAGS=-g -O3
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #31 from Jakub Jelinek jakub at gcc dot gnu.org 2011-02-10
08:43:02 UTC ---
You really don't need to add for_eh argument to output_cfa_loc_raw, you can
assume it is 1.
If you need to still adjust the register numbers inside of
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #32 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-10
14:38:22 UTC ---
(In reply to comment #31)
I don't see how we can directly use DWARF2_FRAME_REG_OUT in
output_loc_operands, except as a template for the desired
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #33 from Iain Sandoe iains at gcc dot gnu.org 2011-02-10 14:39:46
UTC ---
in addition to Mike's fix,
the dw_loc_descr_ref reg numbers need to be adjusted in the loc lists,
below is a hack which works - I don't know the code well
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #34 from Jakub Jelinek jakub at gcc dot gnu.org 2011-02-10
15:01:44 UTC ---
For #c32, you obviously can't hardcode darwin ugly hacks into generic code.
The #c33 patch looks much better, but still
1) in output_loc_sequence, you need to
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #35 from Iain Sandoe iains at gcc dot gnu.org 2011-02-10 15:59:27
UTC ---
(In reply to comment #34)
For #c32, you obviously can't hardcode darwin ugly hacks into generic code.
The #c33 patch looks much better, but still
1) in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #36 from Jakub Jelinek jakub at gcc dot gnu.org 2011-02-10
16:04:46 UTC ---
Yes (though, I'm not a dwarf2out maintainer, so you'll need to ask for
approval).
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #37 from Iain Sandoe iains at gcc dot gnu.org 2011-02-10 17:40:57
UTC ---
(In reply to comment #36)
Yes (though, I'm not a dwarf2out maintainer, so you'll need to ask for
approval).
unfortunately, this causes problems for some PPC
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
CC||tromey at redhat
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #39 from Iain Sandoe iains at gcc dot gnu.org 2011-02-10 18:29:46
UTC ---
(In reply to comment #38)
Wonder what do the debug consumers on PowerPC do currently...
If they actually have separate register number decoders for
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #40 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-10
18:48:07 UTC ---
What happens if you replace all the instances of DWARF2_FRAME_REG_OUT in the
proposed patch with a new DWARF2_FRAME_REG_OUT_EH macro that is a no-op on
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #41 from Jakub Jelinek jakub at gcc dot gnu.org 2011-02-10
18:49:08 UTC ---
Ugh, it is a mess. DW_CFA_expression/DW_CFA_def_cfa_expression location list
comes from 3 different routines:
1) build_cfa_loc
2) build_cfa_aligned_loc
3)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #42 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-10
19:55:37 UTC ---
The Apple linker maintainer says as far as he knows just i386 has the register
numbering schism.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #43 from Iain Sandoe iains at gcc dot gnu.org 2011-02-10 20:13:59
UTC ---
Created attachment 23302
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=23302
tri-stated output_loc_sequence
I agree it's ugly, we need to find a Better
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #44 from Iain Sandoe iains at gcc dot gnu.org 2011-02-10 20:16:42
UTC ---
(In reply to comment #43)
Created attachment 23302 [details]
tri-stated output_loc_sequence
I agree it's ugly, we need to find a Better Way for 4.7 ...
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #45 from Jakub Jelinek jakub at gcc dot gnu.org 2011-02-10
20:24:54 UTC ---
The raw stuff in the patch looks ok to me, it is always used just in the for_eh
case. The for_eh argument to output_loc_operands should be still also *_skip
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
Iain Sandoe iains at gcc dot gnu.org changed:
What|Removed |Added
Attachment #23302|0 |1
is
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #47 from Dominique d'Humieres dominiq at lps dot ens.fr
2011-02-11 07:47:52 UTC ---
Created attachment 23303 [details]
tri-state output_loc_sequence V2
I have updated revisions 170022 on x86_64-apple-darwin10.6.0 and
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
CC||jakub at gcc dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #23 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-09
12:31:27 UTC ---
Any comments on this bit of Mike's patch...
+int
+darwin_dbx_register_number(n)
+{
+#if 1
+ /* Without -O3, eh-alloc-1.c -m32 fails. */
+ /* Works!
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #24 from Ian Lance Taylor ian at airs dot com 2011-02-09 14:10:25
UTC ---
The first branch of darwin_dbx_register_number looks like it will use the wrong
register numbers in debug info. The second branch looks like it will do the
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #25 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-09
14:25:23 UTC ---
Ian, does that mean you believe we should just need...
int
darwin_dbx_register_number(n)
return dbx_register_map[n];
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #26 from Ian Lance Taylor ian at airs dot com 2011-02-09 15:13:22
UTC ---
I think the patch to dwarf2out.c is all you need and I don't understand why you
are thinking about any patch to config/i386/darwin.h and config/i386/darwin.c
at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #27 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-09
16:28:55 UTC ---
Ian,
Just using...
Index: gcc/dwarf2out.c
===
--- gcc/dwarf2out.c(revision
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #28 from Ian Lance Taylor ian at airs dot com 2011-02-09 16:53:22
UTC ---
Then let's find out why that is.
The patch in comment #23 looks clearly incorrect to me.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #29 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-10
02:18:36 UTC ---
(In reply to comment #28)
Then let's find out why that is.
The patch in comment #23 looks clearly incorrect to me.
Okay. The problem with the patch
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #30 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-10
03:03:21 UTC ---
Don't we have to change cfi-dw_cfi_oprnd2.dw_cfi_loc before it is assigned to
loc in output_cfa_loc when for_eh?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #18 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-09
02:03:46 UTC ---
It may be...
--- trunk/gcc/dwarf2out.c2010/03/26 19:21:13157761
+++ trunk/gcc/dwarf2out.c2010/03/26 20:53:58157762
@@ -1115,8 +1115,8
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #19 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-09
02:50:21 UTC ---
Could this be the problem code?
static void
output_cfa_loc (dw_cfi_ref cfi)
{
dw_loc_descr_ref loc;
unsigned long size;
if (cfi-dw_cfi_opc ==
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
--- Comment #20 from Jack Howarth howarth at nitro dot med.uc.edu 2011-02-09
05:10:26 UTC ---
Tested...
Index: gcc/dwarf2out.c
===
--- gcc/dwarf2out.c(revision 169960)
+++
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47324
Ian Lance Taylor ian at airs dot com changed:
What|Removed |Added
CC||ian at airs dot com
39 matches
Mail list logo
