Changeset 11464 in vbox for trunk

Timestamp:

Aug 18, 2008 1:19:26 PM (16 years ago)

Author:

vboxsync

Message:

Iz means sign extend to 64 bits, not zero extend.

Location:

trunk/src/VBox/Disassembler

Files:

• DisasmCore.cpp (modified) (1 diff)
• DisasmTestA.asm (modified) (1 diff)

trunk/src/VBox/Disassembler/DisasmCore.cpp

@@ -1324 +1324 @@
     else
     {
-        pParam->parval = DISReadDWord(pCpu, lpszCodeBlock);
-        /* 64 bits op mode means zero extend to 64 bits. */
+        /* 64 bits op mode means *sign* extend to 64 bits. */
         if (pCpu->opmode == CPUMODE_64BIT)
         {
+            pParam->parval = (uint64_t)(int32_t)DISReadDWord(pCpu, lpszCodeBlock);           
             pParam->flags |= USE_IMMEDIATE64;
             pParam->size   = sizeof(uint64_t);
+            disasmAddStringF(pParam->szParam, sizeof(pParam->szParam), "0%VX64h", pParam->parval);
         }
         else
         {
+            pParam->parval = DISReadDWord(pCpu, lpszCodeBlock);
             pParam->flags |= USE_IMMEDIATE32;
             pParam->size   = sizeof(uint32_t);
-        }
-
-        disasmAddStringF(pParam->szParam, sizeof(pParam->szParam), "0%08Xh", (uint32_t)pParam->parval);
+            disasmAddStringF(pParam->szParam, sizeof(pParam->szParam), "0%08Xh", (uint32_t)pParam->parval);
+        }
         return sizeof(uint32_t);
     }

trunk/src/VBox/Disassembler/DisasmTestA.asm

@@ -71 +71 @@
 align 16
 BEGINPROC TestProc64
+      db 48h
+      db 0c7h
+      db 42h
+      db 18h
+      db 20h
+      db 3eh
+      db 23h
+      db 80h
       call qword [r8+10h]
       ; test