target-alpha: fix cmpbge instruction
authoraurel32 <aurel32@c046a42c-6fe2-441c-8c8c-71466251a162>
Mon, 10 Nov 2008 11:10:14 +0000 (11:10 +0000)
committeraurel32 <aurel32@c046a42c-6fe2-441c-8c8c-71466251a162>
Mon, 10 Nov 2008 11:10:14 +0000 (11:10 +0000)
The cmpbge instruction should compare all 8 bytes of one 64-bit value with
another.  However, we were looping with a < 7 condition which was skipping
the top byte.  So if we were doing a compare where the top byte was
important, we could get the wrong result (this notably breaks the strlen()
function with certain sized strings).

(Vince Weaver)

git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@5667 c046a42c-6fe2-441c-8c8c-71466251a162

target-alpha/op_helper.c

index 393187e..0fa4fe0 100644 (file)
@@ -330,7 +330,7 @@ uint64_t helper_cmpbge (uint64_t op1, uint64_t op2)
     int i;
 
     res = 0;
-    for (i = 0; i < 7; i++) {
+    for (i = 0; i < 8; i++) {
         opa = op1 >> (i * 8);
         opb = op2 >> (i * 8);
         if (opa >= opb)