[llvm] r255811 - [Hexagon] Misc fixes to r255807

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 16 12:07:04 PST 2015


Author: kparzysz
Date: Wed Dec 16 14:07:04 2015
New Revision: 255811

URL: http://llvm.org/viewvc/llvm-project?rev=255811&view=rev
Log:
 [Hexagon] Misc fixes to r255807

Modified:
    llvm/trunk/lib/Target/Hexagon/HexagonVLIWPacketizer.cpp

Modified: llvm/trunk/lib/Target/Hexagon/HexagonVLIWPacketizer.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Hexagon/HexagonVLIWPacketizer.cpp?rev=255811&r1=255810&r2=255811&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Hexagon/HexagonVLIWPacketizer.cpp (original)
+++ llvm/trunk/lib/Target/Hexagon/HexagonVLIWPacketizer.cpp Wed Dec 16 14:07:04 2015
@@ -462,13 +462,13 @@ static const MachineOperand &getPostIncr
       return MO;
 #else
   if (MI->mayLoad()) {
-    MachineOperand &Op1 = MI->getOperand(1);
+    const MachineOperand &Op1 = MI->getOperand(1);
     // The 2nd operand is always the post increment operand in load.
     assert(Op1.isReg() && "Post increment operand has be to a register.");
     return Op1;
   }
   if (MI->getDesc().mayStore()) {
-    MachineOperand &Op0 = MI->getOperand(0);
+    const MachineOperand &Op0 = MI->getOperand(0);
     // The 1st operand is always the post increment operand in store.
     assert(Op0.isReg() && "Post increment operand has be to a register.");
     return Op0;
@@ -1162,16 +1162,11 @@ bool HexagonPacketizerList::isLegalToPac
     MachineInstr *NextMI = NextMII;
 
     bool secondRegMatch = false;
-    bool maintainNewValueJump = false;
     const MachineOperand &NOp0 = NextMI->getOperand(0);
     const MachineOperand &NOp1 = NextMI->getOperand(1);
 
-    if (NOp1.isReg() && I->getOperand(0).getReg() == NOp1.getReg()) {
+    if (NOp1.isReg() && I->getOperand(0).getReg() == NOp1.getReg())
       secondRegMatch = true;
-      maintainNewValueJump = true;
-    } else if (I->getOperand(0).getReg() == NOp0.getReg()) {
-      maintainNewValueJump = true;
-    }
 
     for (auto I : CurrentPacketMIs) {
       SUnit *PacketSU = MIToSUnit.find(I)->second;




More information about the llvm-commits mailing list