On 6/9/22 08:35, Peter Maydell wrote:
+        if (!disas_sme(s, insn)) {
+            unallocated_encoding(s);
+        }
+        break;

I still think we should check bit 31 here.

We don't do anything similar over in a32, where we've done the full conversion:

        if (disas_a32_uncond(s, insn) ||

            disas_vfp_uncond(s, insn) ||

            disas_neon_dp(s, insn) ||

            disas_neon_ls(s, insn) ||

            disas_neon_shared(s, insn)) {

            return;

        }


and there's nothing else within major opcode 0 to conflict.

But if you insist, ok.


r~

Reply via email to