Fix PR9962 by properly constraining register classes in RemoveCopyByCommutingDef...
authorCameron Zwarich <zwarich@apple.com>
Fri, 20 May 2011 23:25:36 +0000 (23:25 +0000)
committerCameron Zwarich <zwarich@apple.com>
Fri, 20 May 2011 23:25:36 +0000 (23:25 +0000)
actually fixes most of the VerifyCoalescing failures in test-suite.

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

lib/CodeGen/SimpleRegisterCoalescing.cpp

index c33ce24c5e8bb2534fd75a6ae4cf5d31a54d7235..21184895ccbd56033fc8477bc6179168689b56c0 100644 (file)
@@ -430,6 +430,10 @@ bool SimpleRegisterCoalescing::RemoveCopyByCommutingDef(const CoalescerPair &CP,
   MachineInstr *NewMI = tii_->commuteInstruction(DefMI);
   if (!NewMI)
     return false;
+  if (TargetRegisterInfo::isVirtualRegister(IntA.reg) &&
+      TargetRegisterInfo::isVirtualRegister(IntB.reg) &&
+      !mri_->constrainRegClass(IntB.reg, mri_->getRegClass(IntA.reg)))
+    return false;
   if (NewMI != DefMI) {
     li_->ReplaceMachineInstrInMaps(DefMI, NewMI);
     MBB->insert(DefMI, NewMI);