[PATCH] D28468: [X86] Fix for bugzilla 31576 - add support for "data32" instruction prefix
Marina Yatsina via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 17 11:50:41 PST 2017
myatsina added a comment.
In https://reviews.llvm.org/D28468#648538, @niravd wrote:
> In https://reviews.llvm.org/D28468#648513, @myatsina wrote:
> > Can you see the tests? I've added comments next to them.
> > Do this tests capture what you meant?
> But I thought you also had one using llvm-objdump so we also test the object->instruction disassembly path.
The llvm-objdump tests are the newly added data-prefix16.s, data-prefix32.s, data-prefix64.s which have the following run line:
RUN: llvm-mc -triple<triple> -filetype=obj %s -o - | llvm-objdump -triple <triple> -d - | FileCheck %s
I will add to those tests the "data16"/"data32" disassembly once I tweak the asm printer.
More information about the llvm-commits