capstone icon indicating copy to clipboard operation
capstone copied to clipboard

SystemZ SYSZ_OP_ACREG can be removed

Open david942j opened this issue 1 year ago • 1 comments

It's not really a bug but more like a tech-debt that after PR #1306, all registers in operands are considered as in the same family SYSZ_OP_REG even for the Access Registers. This fact makes the existence of SYSZ_OP_ACREG confusing. One may check the operands against SYSZ_OP_ACREG but it would never match.

From cstool on v4, d7e459d026b19d6c3a7b743bfc475d919ff03f74

cstool/cstool -d sysz 'b24f0078'
 0  b2 4f 00 78        ear      %r7, %a8
        ID: 383 (ear)
        op_count: 2
                operands[0].type: REG = 7
                operands[1].type: ACREG = 8 // <- reports type in SYSZ_OP_ACREG

on v5, 7905571be70b20574331fe25131ad923d64745ab:

cstool/cstool -d sysz 'b24f0078'
 0  b2 4f 00 78        ear      %r7, %a8
        ID: 383 (ear)
        op_count: 2
                operands[0].type: REG = 7
                operands[1].type: REG = a8 // <- type becomes SYSZ_OP_REG

To address this issue the macro SYSZ_OP_ACREG should be remove to prevent confusions.

david942j avatar Aug 03 '24 13:08 david942j

You can open a PR if you want. But the SystemZ module is update next after the testing. So it will be gone soon anyways.

Rot127 avatar Aug 04 '24 04:08 Rot127