This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Use per instruction prefix control.


On 06 Apr 08:57, H.J. Lu wrote:
> 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.
>
Thanks.

I've committed attached version

Attachment: patch.tar.gz
Description: application/gzip


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]