On 29/08/18 15:25, Jan Beulich wrote: > @@ -2818,6 +2818,9 @@ x86_decode( > > opcode |= b | MASK_INSR(vex.pfx, X86EMUL_OPC_PFX_MASK); > > + if ( !evex.mbs )
This use of mbs is very confusing to read. How about: #define evex_encoded evex.mbs which at least gives a semantic name to the how you are using the mbs bit? ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel