[llvm-commits] [llvm] r78250 - /llvm/trunk/lib/Target/X86/X86InstrInfo.cpp

[Dan Gohman]

Author: djg
Date: Wed Aug  5 17:18:26 2009
New Revision: 78250

URL: http://llvm.org/viewvc/llvm-project?rev=78250&view=rev
Log:
Use GR32 for copies between GR32_NOSP and GR32_NOREX, as neither
is a subset of the other, but both are subsets of GR32.

Modified:
    llvm/trunk/lib/Target/X86/X86InstrInfo.cpp

Modified: llvm/trunk/lib/Target/X86/X86InstrInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86InstrInfo.cpp?rev=78250&r1=78249&r2=78250&view=diff

==============================================================================
--- llvm/trunk/lib/Target/X86/X86InstrInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86InstrInfo.cpp Wed Aug  5 17:18:26 2009
@@ -1697,10 +1697,18 @@
     CommonRC = SrcRC;
   else if (!DestRC->hasSubClass(SrcRC)) {
     // Neither of GR64_NOREX or GR64_NOSP is a superclass of the other,
-    // but we want to copy then as GR64.
-    if ((SrcRC == &X86::GR64RegClass || SrcRC->hasSuperClass(&X86::GR64RegClass)) &&
-        (DestRC == &X86::GR64RegClass || DestRC->hasSuperClass(&X86::GR64RegClass)))
+    // but we want to copy then as GR64. Similarly, for GR32_NOREX and
+    // GR32_NOSP, copy as GR32.
+    if ((SrcRC == &X86::GR64RegClass ||
+         SrcRC->hasSuperClass(&X86::GR64RegClass)) &&
+        (DestRC == &X86::GR64RegClass ||
+         DestRC->hasSuperClass(&X86::GR64RegClass)))
       CommonRC = &X86::GR64RegClass;
+    else if ((SrcRC == &X86::GR32RegClass ||
+              SrcRC->hasSuperClass(&X86::GR32RegClass)) &&
+             (DestRC == &X86::GR32RegClass ||
+              DestRC->hasSuperClass(&X86::GR32RegClass)))
+      CommonRC = &X86::GR32RegClass;
     else
       CommonRC = 0;
   }