[Bug binutils/26705] MODRM isn't properly handled

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=26705

--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot 
gnu.org> ---
The master branch has been updated by H.J. Lu <hjl@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=0e9f3bf12616b108682bb6f6e2a5ef04df9586a8

commit 0e9f3bf12616b108682bb6f6e2a5ef04df9586a8
Author: H.J. Lu <hjl.tools@gmail.com>
Date:   Mon Oct 5 05:23:29 2020 -0700

    x86: Clear modrm if not needed

    The MODRM byte can be checked to display the instruction name only if the
    MODRM byte needed.  Clear modrm if the MODRM byte isn't needed so that
    modrm field checks in putop like, modrm.mod == N with N != 0, can be done
    without checking need_modrm.

    gas/

            PR binutils/26705
            * testsuite/gas/i386/x86-64-suffix.s: Add "mov %rsp,%rbp" before
            sysretq.
            * testsuite/gas/i386/x86-64-suffix-intel.d: Updated.
            * testsuite/gas/i386/x86-64-suffix.d: Likewise.

    opcodes/

            PR binutils/26705
            * i386-dis.c (print_insn): Clear modrm if not needed.
            (putop): Check need_modrm for modrm.mod != 3.  Don't check
            need_modrm for modrm.mod == 3.

-- 
You are receiving this mail because:
You are on the CC list for the bug.