[llvm-commits] [llvm] r61263 - /llvm/trunk/lib/CodeGen/PostRASchedulerList.cpp

[Dan Gohman]

Author: djg
Date: Fri Dec 19 16:23:43 2008
New Revision: 61263

URL: http://llvm.org/viewvc/llvm-project?rev=61263&view=rev
Log:
Use ~0u instead of -1u as the special value, to hopefully avoid
warnings on compilers that warn about such things.

Modified:
    llvm/trunk/lib/CodeGen/PostRASchedulerList.cpp

Modified: llvm/trunk/lib/CodeGen/PostRASchedulerList.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/PostRASchedulerList.cpp?rev=61263&r1=61262&r2=61263&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/PostRASchedulerList.cpp (original)
+++ llvm/trunk/lib/CodeGen/PostRASchedulerList.cpp Fri Dec 19 16:23:43 2008
@@ -217,11 +217,11 @@
   // Map registers to all their references within a live range.
   std::multimap<unsigned, MachineOperand *> RegRefs;
 
-  // The index of the most recent kill (proceding bottom-up), or -1 if
+  // The index of the most recent kill (proceding bottom-up), or ~0u if
   // the register is not live.
   unsigned KillIndices[TargetRegisterInfo::FirstVirtualRegister];
-  std::fill(KillIndices, array_endof(KillIndices), -1);
-  // The index of the most recent complete def (proceding bottom up), or -1 if
+  std::fill(KillIndices, array_endof(KillIndices), ~0u);
+  // The index of the most recent complete def (proceding bottom up), or ~0u if
   // the register is live.
   unsigned DefIndices[TargetRegisterInfo::FirstVirtualRegister];
   std::fill(DefIndices, array_endof(DefIndices), BB->size());
@@ -234,13 +234,13 @@
       unsigned Reg = *I;
       Classes[Reg] = reinterpret_cast<TargetRegisterClass *>(-1);
       KillIndices[Reg] = BB->size();
-      DefIndices[Reg] = -1;
+      DefIndices[Reg] = ~0u;
       // Repeat, for all aliases.
       for (const unsigned *Alias = TRI->getAliasSet(Reg); *Alias; ++Alias) {
         unsigned AliasReg = *Alias;
         Classes[AliasReg] = reinterpret_cast<TargetRegisterClass *>(-1);
         KillIndices[AliasReg] = BB->size();
-        DefIndices[AliasReg] = -1;
+        DefIndices[AliasReg] = ~0u;
       }
     }
   else
@@ -252,13 +252,13 @@
         unsigned Reg = *I;
         Classes[Reg] = reinterpret_cast<TargetRegisterClass *>(-1);
         KillIndices[Reg] = BB->size();
-        DefIndices[Reg] = -1;
+        DefIndices[Reg] = ~0u;
         // Repeat, for all aliases.
         for (const unsigned *Alias = TRI->getAliasSet(Reg); *Alias; ++Alias) {
           unsigned AliasReg = *Alias;
           Classes[AliasReg] = reinterpret_cast<TargetRegisterClass *>(-1);
           KillIndices[AliasReg] = BB->size();
-          DefIndices[AliasReg] = -1;
+          DefIndices[AliasReg] = ~0u;
         }
       }
 
@@ -275,13 +275,13 @@
     unsigned Reg = *I;
     Classes[Reg] = reinterpret_cast<TargetRegisterClass *>(-1);
     KillIndices[Reg] = BB->size();
-    DefIndices[Reg] = -1;
+    DefIndices[Reg] = ~0u;
     // Repeat, for all aliases.
     for (const unsigned *Alias = TRI->getAliasSet(Reg); *Alias; ++Alias) {
       unsigned AliasReg = *Alias;
       Classes[AliasReg] = reinterpret_cast<TargetRegisterClass *>(-1);
       KillIndices[AliasReg] = BB->size();
-      DefIndices[AliasReg] = -1;
+      DefIndices[AliasReg] = ~0u;
     }
   }
 
@@ -458,11 +458,11 @@
         if (NewReg == LastNewReg[AntiDepReg]) continue;
         // If NewReg is dead and NewReg's most recent def is not before
         // AntiDepReg's kill, it's safe to replace AntiDepReg with NewReg.
-        assert(((KillIndices[AntiDepReg] == -1u) != (DefIndices[AntiDepReg] == -1u)) &&
+        assert(((KillIndices[AntiDepReg] == ~0u) != (DefIndices[AntiDepReg] == ~0u)) &&
                "Kill and Def maps aren't consistent for AntiDepReg!");
-        assert(((KillIndices[NewReg] == -1u) != (DefIndices[NewReg] == -1u)) &&
+        assert(((KillIndices[NewReg] == ~0u) != (DefIndices[NewReg] == ~0u)) &&
                "Kill and Def maps aren't consistent for NewReg!");
-        if (KillIndices[NewReg] == -1u &&
+        if (KillIndices[NewReg] == ~0u &&
             Classes[NewReg] != reinterpret_cast<TargetRegisterClass *>(-1) &&
             KillIndices[AntiDepReg] <= DefIndices[NewReg]) {
           DOUT << "Breaking anti-dependence edge on "
@@ -488,7 +488,7 @@
 
           Classes[AntiDepReg] = 0;
           DefIndices[AntiDepReg] = KillIndices[AntiDepReg];
-          KillIndices[AntiDepReg] = -1;
+          KillIndices[AntiDepReg] = ~0u;
 
           RegRefs.erase(AntiDepReg);
           Changed = true;
@@ -511,7 +511,7 @@
       if (MI->isRegReDefinedByTwoAddr(i)) continue;
 
       DefIndices[Reg] = Count;
-      KillIndices[Reg] = -1;
+      KillIndices[Reg] = ~0u;
       Classes[Reg] = 0;
       RegRefs.erase(Reg);
       // Repeat, for all subregs.
@@ -519,7 +519,7 @@
            *Subreg; ++Subreg) {
         unsigned SubregReg = *Subreg;
         DefIndices[SubregReg] = Count;
-        KillIndices[SubregReg] = -1;
+        KillIndices[SubregReg] = ~0u;
         Classes[SubregReg] = 0;
         RegRefs.erase(SubregReg);
       }
@@ -550,21 +550,21 @@
       RegRefs.insert(std::make_pair(Reg, &MO));
 
       // It wasn't previously live but now it is, this is a kill.
-      if (KillIndices[Reg] == -1u) {
+      if (KillIndices[Reg] == ~0u) {
         KillIndices[Reg] = Count;
-        DefIndices[Reg] = -1u;
+        DefIndices[Reg] = ~0u;
       }
       // Repeat, for all aliases.
       for (const unsigned *Alias = TRI->getAliasSet(Reg); *Alias; ++Alias) {
         unsigned AliasReg = *Alias;
-        if (KillIndices[AliasReg] == -1u) {
+        if (KillIndices[AliasReg] == ~0u) {
           KillIndices[AliasReg] = Count;
-          DefIndices[AliasReg] = -1u;
+          DefIndices[AliasReg] = ~0u;
         }
       }
     }
   }
-  assert(Count == -1u && "Count mismatch!");
+  assert(Count == ~0u && "Count mismatch!");
 
   return Changed;
 }