Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* DW_FORM_data8
@ 2002-03-14 13:31 Richard Henderson
  2002-03-14 15:29 ` DW_FORM_data8 Andrew Cagney
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Henderson @ 2002-03-14 13:31 UTC (permalink / raw)
  To: gdb-patches

The dwarf2 reader doesn't believe that DW_FORM_data8 holds
a constant value.  Fixed thus.

Ok?


r~

PS: Not sure what generated this form -- certainly gcc should be
emitting the smaller forms when the array bounds allow.  Perhaps
it's output from an older gcc...  Anyway, it shows up on an ia64
installation I have here.


	* dwarf2read.c (read_array_type): Accept DW_FORM_data8 as
	a constant array bound.

Index: dwarf2read.c
===================================================================
RCS file: /cvs/src/src/gdb/dwarf2read.c,v
retrieving revision 1.49
diff -c -p -d -r1.49 dwarf2read.c
*** dwarf2read.c	2002/02/28 11:21:16	1.49
--- dwarf2read.c	2002/03/14 21:28:59
*************** read_array_type (struct die_info *die, s
*** 2538,2544 ****
  	      else if (attr->form == DW_FORM_udata
  		       || attr->form == DW_FORM_data1
  		       || attr->form == DW_FORM_data2
! 		       || attr->form == DW_FORM_data4)
  		{
  		  low = DW_UNSND (attr);
  		}
--- 2538,2545 ----
  	      else if (attr->form == DW_FORM_udata
  		       || attr->form == DW_FORM_data1
  		       || attr->form == DW_FORM_data2
! 		       || attr->form == DW_FORM_data4
! 		       || attr->form == DW_FORM_data8)
  		{
  		  low = DW_UNSND (attr);
  		}
*************** read_array_type (struct die_info *die, s
*** 2564,2570 ****
  	      else if (attr->form == DW_FORM_udata
  		       || attr->form == DW_FORM_data1
  		       || attr->form == DW_FORM_data2
! 		       || attr->form == DW_FORM_data4)
  		{
  		  high = DW_UNSND (attr);
  		}
--- 2565,2572 ----
  	      else if (attr->form == DW_FORM_udata
  		       || attr->form == DW_FORM_data1
  		       || attr->form == DW_FORM_data2
! 		       || attr->form == DW_FORM_data4
! 		       || attr->form == DW_FORM_data8)
  		{
  		  high = DW_UNSND (attr);
  		}


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: DW_FORM_data8
  2002-03-14 15:29 ` DW_FORM_data8 Andrew Cagney
@ 2002-03-14 14:49   ` Richard Henderson
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Henderson @ 2002-03-14 14:49 UTC (permalink / raw)
  To: Andrew Cagney; +Cc: gdb-patches

On Thu, Mar 14, 2002 at 05:36:20PM -0500, Andrew Cagney wrote:
> Can I suggest adding yourself to the gdb/MAINTAINERS file (write after 
> approval) as an ``obvious fix''.  Don't forget to post the patch.

Done.


r~


Index: MAINTAINERS
===================================================================
RCS file: /cvs/src/src/gdb/MAINTAINERS,v
retrieving revision 1.161
diff -c -p -d -r1.161 MAINTAINERS
*** MAINTAINERS	2002/03/11 10:37:52	1.161
--- MAINTAINERS	2002/03/14 22:48:08
*************** Richard Earnshaw				rearnsha@arm.com
*** 356,361 ****
--- 356,362 ----
  Matthew Green					mrg@eterna.com.au
  Orjan Friberg					orjanf@axis.com
  Ben Harris					bjh21@netbsd.org
+ Richard Henderson				rth@redhat.com
  Paul Hilfinger					hilfinger@gnat.com
  Matt Hiller					hiller@redhat.com
  Kazu Hirata					kazu@hxi.com


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: DW_FORM_data8
  2002-03-14 13:31 DW_FORM_data8 Richard Henderson
@ 2002-03-14 15:29 ` Andrew Cagney
  2002-03-14 14:49   ` DW_FORM_data8 Richard Henderson
  0 siblings, 1 reply; 3+ messages in thread
From: Andrew Cagney @ 2002-03-14 15:29 UTC (permalink / raw)
  To: Richard Henderson; +Cc: gdb-patches

Can I suggest adding yourself to the gdb/MAINTAINERS file (write after 
approval) as an ``obvious fix''.  Don't forget to post the patch.

Please don't ask me about dwarf2 :-)

enjoy,
Andrew


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2002-03-14 23:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-03-14 13:31 DW_FORM_data8 Richard Henderson
2002-03-14 15:29 ` DW_FORM_data8 Andrew Cagney
2002-03-14 14:49   ` DW_FORM_data8 Richard Henderson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox