Centralized all the code used to decode instructions.

git-svn-id: svn://svn.gna.org/svn/chrysalide/trunk@32 abbe820e-26c8-41b2-8c08-b7b2b41f8b0a

1 files changed, 3 insertions, 33 deletions

diff --git a/src/arch/x86/op_adc.c b/src/arch/x86/op_adc.c
index 9ff0f63..8927607 100644
--- a/src/arch/x86/op_adc.c
+++ b/src/arch/x86/op_adc.c
@@ -46,53 +46,23 @@
 *                                                                             *
 ******************************************************************************/
 
-asm_x86_instr *x86_read_instr_adc8_with_reg1632(const uint8_t *data, off_t *pos, off_t len, uint64_t offset, const asm_x86_processor *proc)
+asm_x86_instr *x86_read_instr_adc_imm8_to_rm1632(const uint8_t *data, off_t *pos, off_t len, uint64_t offset, const asm_x86_processor *proc)
 {
     asm_x86_instr *result;                  /* Instruction à retourner     */
     AsmOperandSize oprsize;                 /* Taille des opérandes        */
-    asm_x86_operand *reg;                   /* Registre de destination     */
-    asm_x86_operand *value;                 /* Valeur empilée              */
 
     result = (asm_x86_instr *)calloc(1, sizeof(asm_x86_instr));
 
-    /* Utilisation des registres 32 bits ? */
-    if (data[*pos] == 0x66)
-    {
-        oprsize = switch_x86_operand_size(proc);
-        (*pos)++;
-    }
-    else oprsize = get_x86_current_operand_size(proc);
+    oprsize = switch_x86_operand_size_if_needed(proc, data, pos);
 
     ASM_INSTRUCTION(result)->opcode = data[(*pos)++];
 
-    /* TODO ! */
-    if ((data[*pos] & 0xc0) != 0xc0)
-        return NULL;
-
-    reg = x86_create_reg1632_operand_from_modrm(data[*pos], oprsize == AOS_32_BITS, false);
-    if (reg == NULL)
-    {
-        free(result);
-        return NULL;
-    }
-
-    (*pos)++;
-
-    value = create_new_x86_operand();
-    if (!fill_imm_operand(ASM_OPERAND(value), AOS_8_BITS, data, pos, len))
+    if (!x86_read_two_operands(result, data, pos, len, X86_OTP_RM1632, X86_OTP_IMM8, oprsize))
     {
-        free(reg);
-        free(value);
         free(result);
         return NULL;
     }
 
-    ASM_INSTRUCTION(result)->operands = (asm_operand **)calloc(2, sizeof(asm_operand *));
-    ASM_INSTRUCTION(result)->operands_count = 2;
-
-    ASM_INSTRUCTION(result)->operands[0] = ASM_OPERAND(reg);
-    ASM_INSTRUCTION(result)->operands[1] = ASM_OPERAND(value);
-
     return result;
 
 }