From: Andreas Schwab <schwab@linux-m68k.org>
Date: Thu, 11 Aug 2005 13:45:40 +0000 (+0000)
Subject: 	PR exp/1978
X-Git-Tag: gdb_6_4-branchpoint~488
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=0d059fcaa4840fdd24f0664318dc3d36218242e1;p=thirdparty%2Fbinutils-gdb.git

	PR exp/1978
	* valarith.c (value_binop): Fix result type for the shift
	operators.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index b5d805d1016..637919add06 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2005-08-11  Andreas Schwab  <schwab@suse.de>
+
+	PR exp/1978
+	* valarith.c (value_binop): Fix result type for the shift
+	operators.
+
 2005-08-09  Mark Kettenis  <kettenis@gnu.org>
 
 	* sparc64obsd-tdep.c: Tweak comments.
diff --git a/gdb/valarith.c b/gdb/valarith.c
index ef03fb9839c..2e1471c3bc4 100644
--- a/gdb/valarith.c
+++ b/gdb/valarith.c
@@ -883,7 +883,14 @@ value_binop (struct value *arg1, struct value *arg2, enum exp_opcode op)
          Use the signedness of the operand with the greater length.
          If both operands are of equal length, use unsigned operation
          if one of the operands is unsigned.  */
-      if (promoted_len1 > promoted_len2)
+      if (op == BINOP_RSH || op == BINOP_LSH)
+	{
+	  /* In case of the shift operators the type of the result only
+	     depends on the type of the left operand.  */
+	  unsigned_operation = is_unsigned1;
+	  result_len = promoted_len1;
+	}
+      else if (promoted_len1 > promoted_len2)
 	{
 	  unsigned_operation = is_unsigned1;
 	  result_len = promoted_len1;