fix an encoding mismatch where "sal %eax, 1" was not using the short encoding

for shl.  Caught by inspection.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@117820 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Target/X86/AsmParser/X86AsmParser.cpp b/lib/Target/X86/AsmParser/X86AsmParser.cpp
index 8a7cdc788ab37edf12428c43dbd29c954481aec9..e92441682cfea8664df9968384293e53252f5614 100644 (file)
--- a/lib/Target/X86/AsmParser/X86AsmParser.cpp
+++ b/lib/Target/X86/AsmParser/X86AsmParser.cpp
@@ -783,7 +783,7 @@ ParseInstruction(StringRef Name, SMLoc NameLoc,
   // FIXME: Hack to handle recognize s{hr,ar,hl} $1, <op>.  Canonicalize to
   // "shift <op>".
   if ((Name.startswith("shr") || Name.startswith("sar") ||
-       Name.startswith("shl")) &&
+       Name.startswith("shl") || Name.startswith("sal")) &&
       Operands.size() == 3) {
     X86Operand *Op1 = static_cast<X86Operand*>(Operands[1]);
     if (Op1->isImm() && isa<MCConstantExpr>(Op1->getImm()) &&

diff --git a/test/MC/X86/x86-32.s b/test/MC/X86/x86-32.s
index 7407767f7a5f6acd48f61d4c108fc5f71309e2dc..41876333a6b423e3ca5c5c44dd9eb8850894a9f0 100644 (file)
--- a/test/MC/X86/x86-32.s
+++ b/test/MC/X86/x86-32.s
@@ -57,6 +57,11 @@ rdtscp
 // CHECK: shrl %eax                    # encoding: [0xd1,0xe8]
         shrl $1, %eax
 
+// CHECK: shll %eax                    # encoding: [0xd1,0xe0]
+        sall $1, %eax
+// CHECK: shll %eax                    # encoding: [0xd1,0xe0]
+        sal $1, %eax
+
 // moffset forms of moves, rdar://7947184
 movb   0, %al    // CHECK: movb 0, %al  # encoding: [0xa0,A,A,A,A]
 movw   0, %ax    // CHECK: movw 0, %ax  # encoding: [0x66,0xa1,A,A,A,A]