From: Jim Blandy <jimb@codesourcery.com>
Date: Wed, 19 Dec 2001 22:24:49 +0000 (+0000)
Subject: * s390-tdep.c (s390_register_raw_size): Replace unnecessarily
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=23b7362fd74b2ac828e9bb7c0251aaa144948757;p=binutils-gdb.git

* s390-tdep.c (s390_register_raw_size): Replace unnecessarily
obscure range comparison with legible equivalent.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 9091a608948..6d55fa7a19c 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2001-12-19  Jim Blandy  <jimb@redhat.com>
+
+	* s390-tdep.c (s390_register_raw_size): Replace unnecessarily
+	obscure range comparison with legible equivalent.
+
 Wed Dec 19 12:18:57 2001  Jeffrey A Law  (law@redhat.com)
 
 	* config/pa/tm-hppa.h (STORE_RETURN_VALUE): Use hppa_store_return_value.
diff --git a/gdb/s390-tdep.c b/gdb/s390-tdep.c
index ddeacadd7fe..ba8c9a3e314 100644
--- a/gdb/s390-tdep.c
+++ b/gdb/s390-tdep.c
@@ -42,14 +42,15 @@
 
 
 /* Number of bytes of storage in the actual machine representation
-   for register N. 
-   Note that the unsigned cast here forces the result of the
-   subtraction to very high positive values if N < S390_FP0_REGNUM */
+   for register N.  */
 int
 s390_register_raw_size (int reg_nr)
 {
-  return ((unsigned) reg_nr - S390_FP0_REGNUM) <
-    S390_NUM_FPRS ? S390_FPR_SIZE : 4;
+  if (S390_FP0_REGNUM <= reg_nr
+      && reg_nr < S390_FP0_REGNUM + S390_NUM_FPRS)
+    return S390_FPR_SIZE;
+  else
+    return 4;
 }
 
 int
@@ -1759,6 +1760,7 @@ s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
   gdbarch = gdbarch_alloc (&info, NULL);
 
   set_gdbarch_believe_pcc_promotion (gdbarch, 0);
+  set_gdbarch_char_signed (gdbarch, 0);
 
   set_gdbarch_frame_args_skip (gdbarch, 0);
   set_gdbarch_frame_args_address (gdbarch, s390_frame_args_address);