[RFA]: fix for sizeof operator for ia64

[RFA]: fix for sizeof operator for ia64

[J. Johnston]

[-- Attachment #1: Type: text/plain, Size: 270 bytes --]

The attached patch fixes the number of bits for a ia64 long double.
This fixes the sizeof.exp testcase in gdb.base.

Ok to commit?

-- Jeff J.

2003-06-10  Jeff Johnston  <jjohnstn@redhat.com>

	* ia64-tdep.c (ia64_gdbarch_init): Set number of long double bits to 128.


[-- Attachment #2: ia64-tdep.patch --]
[-- Type: text/plain, Size: 644 bytes --]

Index: ia64-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/ia64-tdep.c,v
retrieving revision 1.83
diff -u -p -r1.83 ia64-tdep.c
--- ia64-tdep.c	9 Jun 2003 17:35:57 -0000	1.83
+++ ia64-tdep.c	10 Jun 2003 21:19:07 -0000
@@ -2243,7 +2243,7 @@ ia64_gdbarch_init (struct gdbarch_info i
   set_gdbarch_long_long_bit (gdbarch, 64);
   set_gdbarch_float_bit (gdbarch, 32);
   set_gdbarch_double_bit (gdbarch, 64);
-  set_gdbarch_long_double_bit (gdbarch, 64);
+  set_gdbarch_long_double_bit (gdbarch, 128);
   set_gdbarch_ptr_bit (gdbarch, 64);
 
   set_gdbarch_num_regs (gdbarch, ia64_num_regs);

Re: [RFA]: fix for sizeof operator for ia64

[Mark Kettenis]

"J. Johnston" <jjohnstn@redhat.com> writes:

> The attached patch fixes the number of bits for a ia64 long double.
> This fixes the sizeof.exp testcase in gdb.base.
> 
> Ok to commit?

GCC has #define LONG_DOUBLE_TYPE_SIZE 128, so go for it if you don't
hear from Kevin soon.

Mark

> -- Jeff J.
> 
> 2003-06-10  Jeff Johnston  <jjohnstn@redhat.com>
> 
> 	* ia64-tdep.c (ia64_gdbarch_init): Set number of long double bits to 128.
> 

Re: [RFA]: fix for sizeof operator for ia64

[Kevin Buettner]

On Jun 10,  5:40pm, J. Johnston wrote:

> The attached patch fixes the number of bits for a ia64 long double.
> This fixes the sizeof.exp testcase in gdb.base.
> 
> Ok to commit?
> 
> -- Jeff J.
> 
> 2003-06-10  Jeff Johnston  <jjohnstn@redhat.com>
> 
> 	* ia64-tdep.c (ia64_gdbarch_init): Set number of long double bits to 128.

Okay.  (I verified that your patch is correct by checking the ABI document.)

Make sure that your ChangeLog entry doesn't exceed 80 characters per line
though.

Kevin

Re: [RFA]: fix for sizeof operator for ia64

[J. Johnston]

Patch committed.  ChangeLog entry trimmed as asked.  I always do this
when I copy the ChangeLog entry.

Thanks,

-- Jeff J.

Kevin Buettner wrote:
> On Jun 10,  5:40pm, J. Johnston wrote:
> 
> 
>>The attached patch fixes the number of bits for a ia64 long double.
>>This fixes the sizeof.exp testcase in gdb.base.
>>
>>Ok to commit?
>>
>>-- Jeff J.
>>
>>2003-06-10  Jeff Johnston  <jjohnstn@redhat.com>
>>
>>	* ia64-tdep.c (ia64_gdbarch_init): Set number of long double bits to 128.
> 
> 
> Okay.  (I verified that your patch is correct by checking the ABI document.)
> 
> Make sure that your ChangeLog entry doesn't exceed 80 characters per line
> though.
> 
> Kevin
>