[patch] Fix record for lddqu (PR 12361)

Hui Zhu teawater@gmail.com
Tue Feb 8 12:46:00 GMT 2011


Hi Jan,

That is great.  Please go ahead.

Thanks,
Hui

On Mon, Feb 7, 2011 at 00:33, Jan Kratochvil <jan.kratochvil@redhat.com> wrote:
> Hi,
>
> it is generally a blocker of using record on Fedora 14 x86_64, cross-checked
> the opcode in:
>        http://developer.intel.com/products/processor/manuals/
>
> No regressions on {x86_64,x86_64-m32,i686}-fedora14-linux-gnu.
>
> Do you agree with the check-in?
>
>
> Thanks,
> Jan
>
>
> gdb/
> 2011-02-06  Jan Kratochvil  <jan.kratochvil@redhat.com>
>
>        PR 12361.
>        * i386-tdep.c (i386_process_record) <0x660f3807>: Fix the comment to
>        phsubsw.
>        (i386_process_record) <lddqu>: Fix the opcode to 0xf20ff0.
>        (i386_process_record) <0x0f3807>: Fix the comment to phsubsw.
>
> --- a/gdb/i386-tdep.c
> +++ b/gdb/i386-tdep.c
> @@ -6256,7 +6256,7 @@ reswitch_prefix_add:
>         case 0x660f3804:    /* pmaddubsw */
>         case 0x660f3805:    /* phsubw */
>         case 0x660f3806:    /* phsubd */
> -        case 0x660f3807:    /* phaddsw */
> +        case 0x660f3807:    /* phsubsw */
>         case 0x660f3808:    /* psignb */
>         case 0x660f3809:    /* psignw */
>         case 0x660f380a:    /* psignd */
> @@ -6409,7 +6409,7 @@ reswitch_prefix_add:
>         case 0x660fed:      /* paddsw */
>         case 0x660fee:      /* pmaxsw */
>         case 0x660fef:      /* pxor */
> -        case 0x660ff0:      /* lddqu */
> +        case 0xf20ff0:      /* lddqu */
>         case 0x660ff1:      /* psllw */
>         case 0x660ff2:      /* pslld */
>         case 0x660ff3:      /* psllq */
> @@ -6525,7 +6525,7 @@ reswitch_prefix_add:
>         case 0x0f3804:    /* pmaddubsw */
>         case 0x0f3805:    /* phsubw */
>         case 0x0f3806:    /* phsubd */
> -        case 0x0f3807:    /* phaddsw */
> +        case 0x0f3807:    /* phsubsw */
>         case 0x0f3808:    /* psignb */
>         case 0x0f3809:    /* psignw */
>         case 0x0f380a:    /* psignd */
>



More information about the Gdb-patches mailing list