[llvm] r266027 - [RegisterBankInfo] Do not provide a default mapping for non-reg of phi

[Quentin Colombet via llvm-commits]

Author: qcolombet
Date: Mon Apr 11 19:30:14 2016
New Revision: 266027

URL: http://llvm.org/viewvc/llvm-project?rev=266027&view=rev
Log:
[RegisterBankInfo] Do not provide a default mapping for non-reg of phi
operations.

Modified:
    llvm/trunk/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp

Modified: llvm/trunk/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp?rev=266027&r1=266026&r2=266027&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp (original)
+++ llvm/trunk/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp Mon Apr 11 19:30:14 2016
@@ -309,10 +309,17 @@ RegisterBankInfo::getInstrMappingImpl(co
   // Propagate RegBank to all operands that do not have a
   // mapping yet.
   for (unsigned OpIdx = 0, End = MI.getNumOperands(); OpIdx != End; ++OpIdx) {
+    const MachineOperand &MO = MI.getOperand(OpIdx);
+    // Don't assign a mapping for non-reg operands.
+    if (!MO.isReg())
+      continue;
+
+    // If a mapping already exists, do not touch it.
     if (!static_cast<const InstructionMapping *>(&Mapping)
              ->getOperandMapping(OpIdx)
              .BreakDown.empty())
       continue;
+
     Mapping.setOperandMapping(OpIdx, RegSize, *RegBank);
   }
   return Mapping;