This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH/PR14355] Fix in XOP table in objdump
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Michael Zolotukhin <michael dot v dot zolotukhin at gmail dot com>
- Cc: binutils at sourceware dot org, Kirill Yukhin <kirill dot yukhin at gmail dot com>
- Date: Thu, 19 Jul 2012 06:42:24 -0700
- Subject: Re: [PATCH/PR14355] Fix in XOP table in objdump
- References: <CANtU07-Fz7G6EuvDGWZan_HAbhcTz6vqPOYO2xSqo3GbU9dBsw@mail.gmail.com>
On Thu, Jul 19, 2012 at 5:41 AM, Michael Zolotukhin
<michael.v.zolotukhin@gmail.com> wrote:
> Hi,
> The patch fixes this bug:
> http://sourceware.org/bugzilla/show_bug.cgi?id=14355
>
> Ok for trunk?
>
> opcodes/Changelog entry:
> 2012-07-19 Michael Zolotukhin <michael.v.zolotukhin@intel.com>
>
> PR binutils/14355
> * i386-dis.c (VEX_LEN_0FXOP_08_CC): New.
> (VEX_LEN_0FXOP_08_CD): Likewise.
> (VEX_LEN_0FXOP_08_CE): Likewise.
> (VEX_LEN_0FXOP_08_CF): Likewise.
> (VEX_LEN_0FXOP_08_EC): Likewise.
> (VEX_LEN_0FXOP_08_ED): Likewise.
> (VEX_LEN_0FXOP_08_EE): Likewise.
> (VEX_LEN_0FXOP_08_EF): Likewise.
> (xop_table): Fix entries for vpcomb, vpcomw, vpcomd, vpcomq, vpcomub,
> vpcomuw, vpcomud, vpcomuq.
> (vex_len_table): Add entries for VEX_LEN_0FXOP_08_CC,
> VEX_LEN_0FXOP_08_CD, VEX_LEN_0FXOP_08_CE, VEX_LEN_0FXOP_08_CF,
> VEX_LEN_0FXOP_08_EC, VEX_LEN_0FXOP_08_ED, VEX_LEN_0FXOP_08_EE,
> VEX_LEN_0FXOP_08_EF.
>
>
It looks good. I checked it in for you with a tiny change.
Thanks.
--
H.J.