commit 3aef88b1c75316913773f4d0b77cc3ba405300b9
author Craig Topper <craig.topper@gmail.com> Sun Sep 29 08:33:34 2013 +0000
committer Craig Topper <craig.topper@gmail.com> Sun Sep 29 08:33:34 2013 +0000
tree 28fa4edf6a2ae5f7c9501e56b416f1895b49df81
parent d062146896ffa711c1f413b002d964eae4c30e4a

Change type of XOP flag in code emitters to a bool. Remove a some unneeded cases from switch.

llvm-svn: 191632

llvm/lib/Target/X86/X86CodeEmitter.cpp

diff --git a/llvm/lib/Target/X86/X86CodeEmitter.cpp b/llvm/lib/Target/X86/X86CodeEmitter.cpp
index 806c0b1..14385ed 100644
--- a/llvm/lib/Target/X86/X86CodeEmitter.cpp
+++ b/llvm/lib/Target/X86/X86CodeEmitter.cpp
@@ -840,7 +840,7 @@
   unsigned char VEX_W = 0;
 
   // XOP: Use XOP prefix byte 0x8f instead of VEX.
-  unsigned char XOP = 0;
+  bool XOP = false;
 
   // VEX_5M (VEX m-mmmmm field):
   //
@@ -883,7 +883,7 @@
     VEX_W = 1;
 
   if ((TSFlags >> X86II::VEXShift) & X86II::XOP)
-    XOP = 1;
+    XOP = true;
 
   if ((TSFlags >> X86II::VEXShift) & X86II::VEX_L)
     VEX_L = 1;
@@ -923,11 +923,8 @@
     case X86II::XOPA:
       VEX_5M = 0xA;
       break;
-    case X86II::A6:  // Bypass: Not used by VEX
-    case X86II::A7:  // Bypass: Not used by VEX
-    case X86II::TB:  // Bypass: Not used by VEX
-    case 0:
-      break;  // No prefix!
+    case X86II::TB: // VEX_5M/VEX_PP already correct
+      break;
   }