From: Jim Blandy <jimb@codesourcery.com>
Date: Sun, 7 Jan 2007 15:35:44 +0000 (+0000)
Subject: * ax-general.c (ax_const_l): Select proper opcode for the given
X-Git-Tag: drow-reverse-20070409-branchpoint~866
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=44a8177417b268fdcf9d619ae14af64156b2b5f5;p=external%2Fbinutils.git

* ax-general.c (ax_const_l): Select proper opcode for the given
value.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 25dac41..2fa60c1 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2007-01-07  Jim Blandy  <jimb@codesourcery.com>
+
+	* ax-general.c (ax_const_l): Select proper opcode for the given
+	value.
+
 2007-01-04  Vladimir Prus  <vladimir@codesourcery.com>
 
 	* varobj.c (c_value_of_root): Don't select frame if variable
diff --git a/gdb/ax-general.c b/gdb/ax-general.c
index c08272a..8c1dfe1 100644
--- a/gdb/ax-general.c
+++ b/gdb/ax-general.c
@@ -231,8 +231,12 @@ ax_const_l (struct agent_expr *x, LONGEST l)
      signed or unsigned; we always reproduce the value exactly, and
      use the shortest representation.  */
   for (op = 0, size = 8; size < 64; size *= 2, op++)
-    if (-((LONGEST) 1 << size) <= l && l < ((LONGEST) 1 << size))
-      break;
+    {
+      LONGEST lim = 1 << (size - 1);
+
+      if (-lim <= l && l <= lim - 1)
+        break;
+    }
 
   /* Emit the right opcode... */
   ax_simple (x, ops[op]);