correctly print FPU instructions with .p suffix

Author: jack68k

arch/M68K/M68KDisassembler.c

@@ -2371,12 +2371,17 @@ static void d68020_fpu(m68k_info *info)
 			break;
 
 		case 0x02: // extended real
-		case 0x03: // packed decimal
 			ext->op_size.type = M68K_SIZE_TYPE_FPU;
 			ext->op_size.fpu_size = M68K_FPU_SIZE_EXTENDED;
 			get_ea_mode_op(info, op0, info->ir, 12);
 			break;
 
+		case 0x03: // packed decimal
+			ext->op_size.type = M68K_SIZE_TYPE_FPU_PACKED;
+			ext->op_size.fpu_size = M68K_FPU_SIZE_PACKED;
+			get_ea_mode_op(info, op0, info->ir, 12);
+			break;
+
 		default:
 			d68000_invalid(info);
 			return;

arch/M68K/M68KInstPrinter.c

@@ -438,6 +438,10 @@ void M68K_printInst(MCInst *MI, SStream *O, void *PrinterInfo)
 			break;
 		}
 		break;
+
+	case M68K_SIZE_TYPE_FPU_PACKED:
+		SStream_concat0(O, ".p");
+		break;
 	}
 
 	SStream_concat0(O, " ");

include/capstone/m68k.h

@@ -203,22 +203,24 @@ typedef enum m68k_fpu_size {
 	M68K_FPU_SIZE_SINGLE = 4, ///< 4 byte in size (single float)
 	M68K_FPU_SIZE_DOUBLE = 8, ///< 8 byte in size (double)
 	M68K_FPU_SIZE_EXTENDED = 12, ///< 12 byte in size (extended real format)
+	M68K_FPU_SIZE_PACKED = 12, ///< 12 byte in size (packed decimal real format)
 } m68k_fpu_size;
 
 /// Type of size that is being used for the current instruction
 typedef enum m68k_size_type {
 	M68K_SIZE_TYPE_INVALID = 0,
 
 	M68K_SIZE_TYPE_CPU,
-	M68K_SIZE_TYPE_FPU,
+	M68K_SIZE_TYPE_FPU, ///< FPU instructions with .s, .d, and .x suffixes
+	M68K_SIZE_TYPE_FPU_PACKED,  ///< FPU instructions with .p suffix
 } m68k_size_type;
 
-/// Operation size of the current instruction (NOT the actually size of instruction)
+/// Operation size of the current instruction (NOT the actual size of instruction)
 typedef struct m68k_op_size {
 	m68k_size_type type;
 	union {
-		m68k_cpu_size cpu_size;
-		m68k_fpu_size fpu_size;
+		m68k_cpu_size cpu_size; ///< when type is M68K_SIZE_TYPE_CPU
+		m68k_fpu_size fpu_size; ///< when type is M68K_SIZE_TYPE_FPU or M68K_SIZE_TYPE_FPU_PACKED
 	};
 } m68k_op_size;
 

suite/cstest/include/test_mapping.h

@@ -923,6 +923,7 @@ static const cs_enum_id_map cs_enum_map[] = {
 	{ .str = "M68K_OP_REG_PAIR", .val = M68K_OP_REG_PAIR },
 	{ .str = "M68K_SIZE_TYPE_CPU", .val = M68K_SIZE_TYPE_CPU },
 	{ .str = "M68K_SIZE_TYPE_FPU", .val = M68K_SIZE_TYPE_FPU },
+	{ .str = "M68K_SIZE_TYPE_FPU_PACKED", .val = M68K_SIZE_TYPE_FPU_PACKED },
 	{ .str = "MIPS_INS_ADDIU", .val = MIPS_INS_ADDIU },
 	{ .str = "MIPS_INS_ADDIUPC", .val = MIPS_INS_ADDIUPC },
 	{ .str = "MIPS_INS_ALIAS_B", .val = MIPS_INS_ALIAS_B },

tests/details/m68k.yaml

@@ -1291,7 +1291,7 @@ test_cases:
                 type: M68K_OP_REG
                 reg: fp0
       -
-        asm_text: "fmove.x (a0), fp0"
+        asm_text: "fmove.p (a0), fp0"
         details:
           regs_read: [ a0 ]
           regs_write: [ fp0 ]