[PATCH] Use per instruction prefix control.

H.J. Lu hjl.tools@gmail.com
Mon Apr 6 15:57:00 GMT 2015


On Mon, Apr 6, 2015 at 8:46 AM, Ilya Tocar <tocarip.intel@gmail.com> wrote:
> Hi,
>
> To deal with problems like
> https://sourceware.org/bugzilla/show_bug.cgi?id=17898
> We need more fine-grained prefix control than
> twobyte_has_mandatory_prefix[256]
> Attached patch adds mandatory_prefix field to struct dis386,
> which will be used to control allowed/mandatory/forbidden prefixes on
> per instruction basis.
>

A prefix can be optional for an opcode which may be UD
with the other prefixes.  Please remove

#define PREFIX_OPTIONAL 0

and replace

{ "vfpclassss",    { XMask, EXxmm_md, Ib }, PREFIX_OPTIONAL },

with

{ "vfpclassss",    { XMask, EXxmm_md, Ib }, 0 },

OK with this change.

Thanks.

-- 
H.J.



More information about the Binutils mailing list