[llvm] r192620 - Remove extra indentation in LiveRegUnits.

Andrew Trick atrick at apple.com
Mon Oct 14 13:45:11 PDT 2013


Author: atrick
Date: Mon Oct 14 15:45:11 2013
New Revision: 192620

URL: http://llvm.org/viewvc/llvm-project?rev=192620&view=rev
Log:
Remove extra indentation in LiveRegUnits.

Modified:
    llvm/trunk/include/llvm/CodeGen/LiveRegUnits.h

Modified: llvm/trunk/include/llvm/CodeGen/LiveRegUnits.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/LiveRegUnits.h?rev=192620&r1=192619&r2=192620&view=diff
==============================================================================
--- llvm/trunk/include/llvm/CodeGen/LiveRegUnits.h (original)
+++ llvm/trunk/include/llvm/CodeGen/LiveRegUnits.h Mon Oct 14 15:45:11 2013
@@ -24,133 +24,133 @@
 
 namespace llvm {
 
-  class MachineInstr;
+class MachineInstr;
 
-  /// A set of live register units with functions to track liveness when walking
-  /// backward/forward through a basic block.
-  class LiveRegUnits {
-    SmallSet<unsigned, 32> LiveUnits;
-
-  public:
-    /// Constructs a new empty LiveRegUnits set.
-    LiveRegUnits() {
-    }
-
-    /// Constructs a new LiveRegUnits set by copying @p Other.
-    LiveRegUnits(const LiveRegUnits &Other)
-      : LiveUnits(Other.LiveUnits) {
-    }
-
-    /// Adds a register to the set.
-    void addReg(unsigned Reg, const MCRegisterInfo &MCRI) {
-      for (MCRegUnitIterator RUnits(Reg, &MCRI); RUnits.isValid(); ++RUnits)
-        LiveUnits.insert(*RUnits);
-    }
-
-    /// Removes a register from the set.
-    void removeReg(unsigned Reg, const MCRegisterInfo &MCRI) {
-      for (MCRegUnitIterator RUnits(Reg, &MCRI); RUnits.isValid(); ++RUnits)
-        LiveUnits.erase(*RUnits);
-    }
-
-    /// \brief Removes registers clobbered by the regmask operand @p Op.
-    /// Note that we assume the high bits of a physical super register are not
-    /// preserved unless the instruction has an implicit-use operand reading
-    /// the super-register or a register unit for the upper bits is available.
-    void removeRegsInMask(const MachineOperand &Op,
-                          const MCRegisterInfo &MCRI) {
-      const uint32_t *Mask = Op.getRegMask();
-      unsigned Bit = 0;
-      for (unsigned R = 0; R < MCRI.getNumRegs(); ++R) {
-        if ((*Mask & (1u << Bit)) == 0)
-          removeReg(R, MCRI);
-        ++Bit;
-        if (Bit >= 32) {
-          Bit = 0;
-          ++Mask;
-        }
-      }
-    }
-
-    /// Returns true if register @p Reg (or one of its super register) is
-    /// contained in the set.
-    bool contains(unsigned Reg, const MCRegisterInfo &MCRI) const {
-      for (MCRegUnitIterator RUnits(Reg, &MCRI); RUnits.isValid(); ++RUnits) {
-        if (LiveUnits.count(*RUnits))
-          return true;
-      }
-      return false;
-    }
-
-    /// Simulates liveness when stepping backwards over an instruction(bundle):
-    /// Defs are removed from the set, uses added.
-    void stepBackward(const MachineInstr &MI, const MCRegisterInfo &MCRI) {
-      // Remove defined registers and regmask kills from the set.
-      for (ConstMIBundleOperands O(&MI); O.isValid(); ++O) {
-        if (O->isReg()) {
-          if (!O->isDef())
-            continue;
-          unsigned Reg = O->getReg();
-          if (Reg == 0)
-            continue;
-          removeReg(Reg, MCRI);
-        } else if (O->isRegMask()) {
-          removeRegsInMask(*O, MCRI);
-        }
-      }
-      // Add uses to the set.
-      for (ConstMIBundleOperands O(&MI); O.isValid(); ++O) {
-        if (!O->isReg() || !O->readsReg() || O->isUndef())
+/// A set of live register units with functions to track liveness when walking
+/// backward/forward through a basic block.
+class LiveRegUnits {
+  SmallSet<unsigned, 32> LiveUnits;
+
+public:
+  /// Constructs a new empty LiveRegUnits set.
+  LiveRegUnits() {
+  }
+
+  /// Constructs a new LiveRegUnits set by copying @p Other.
+  LiveRegUnits(const LiveRegUnits &Other)
+    : LiveUnits(Other.LiveUnits) {
+  }
+
+  /// Adds a register to the set.
+  void addReg(unsigned Reg, const MCRegisterInfo &MCRI) {
+    for (MCRegUnitIterator RUnits(Reg, &MCRI); RUnits.isValid(); ++RUnits)
+      LiveUnits.insert(*RUnits);
+  }
+
+  /// Removes a register from the set.
+  void removeReg(unsigned Reg, const MCRegisterInfo &MCRI) {
+    for (MCRegUnitIterator RUnits(Reg, &MCRI); RUnits.isValid(); ++RUnits)
+      LiveUnits.erase(*RUnits);
+  }
+
+  /// \brief Removes registers clobbered by the regmask operand @p Op.
+  /// Note that we assume the high bits of a physical super register are not
+  /// preserved unless the instruction has an implicit-use operand reading
+  /// the super-register or a register unit for the upper bits is available.
+  void removeRegsInMask(const MachineOperand &Op,
+                        const MCRegisterInfo &MCRI) {
+    const uint32_t *Mask = Op.getRegMask();
+    unsigned Bit = 0;
+    for (unsigned R = 0; R < MCRI.getNumRegs(); ++R) {
+      if ((*Mask & (1u << Bit)) == 0)
+        removeReg(R, MCRI);
+      ++Bit;
+      if (Bit >= 32) {
+        Bit = 0;
+        ++Mask;
+      }
+    }
+  }
+
+  /// Returns true if register @p Reg (or one of its super register) is
+  /// contained in the set.
+  bool contains(unsigned Reg, const MCRegisterInfo &MCRI) const {
+    for (MCRegUnitIterator RUnits(Reg, &MCRI); RUnits.isValid(); ++RUnits) {
+      if (LiveUnits.count(*RUnits))
+        return true;
+    }
+    return false;
+  }
+
+  /// Simulates liveness when stepping backwards over an instruction(bundle):
+  /// Defs are removed from the set, uses added.
+  void stepBackward(const MachineInstr &MI, const MCRegisterInfo &MCRI) {
+    // Remove defined registers and regmask kills from the set.
+    for (ConstMIBundleOperands O(&MI); O.isValid(); ++O) {
+      if (O->isReg()) {
+        if (!O->isDef())
           continue;
         unsigned Reg = O->getReg();
         if (Reg == 0)
           continue;
-        addReg(Reg, MCRI);
-      }
-    }
-
-    /// \brief Simulates liveness when stepping forward over an
-    /// instruction(bundle).
-    ///
-    /// Uses with kill flag get removed from the set, defs added. If possible
-    /// use StepBackward() instead of this function because some kill flags may
-    /// be missing.
-    void stepForward(const MachineInstr &MI, const MCRegisterInfo &MCRI) {
-      SmallVector<unsigned, 4> Defs;
-      // Remove killed registers from the set.
-      for (ConstMIBundleOperands O(&MI); O.isValid(); ++O) {
-        if (O->isReg()) {
-          unsigned Reg = O->getReg();
-          if (Reg == 0)
+        removeReg(Reg, MCRI);
+      } else if (O->isRegMask()) {
+        removeRegsInMask(*O, MCRI);
+      }
+    }
+    // Add uses to the set.
+    for (ConstMIBundleOperands O(&MI); O.isValid(); ++O) {
+      if (!O->isReg() || !O->readsReg() || O->isUndef())
+        continue;
+      unsigned Reg = O->getReg();
+      if (Reg == 0)
+        continue;
+      addReg(Reg, MCRI);
+    }
+  }
+
+  /// \brief Simulates liveness when stepping forward over an
+  /// instruction(bundle).
+  ///
+  /// Uses with kill flag get removed from the set, defs added. If possible
+  /// use StepBackward() instead of this function because some kill flags may
+  /// be missing.
+  void stepForward(const MachineInstr &MI, const MCRegisterInfo &MCRI) {
+    SmallVector<unsigned, 4> Defs;
+    // Remove killed registers from the set.
+    for (ConstMIBundleOperands O(&MI); O.isValid(); ++O) {
+      if (O->isReg()) {
+        unsigned Reg = O->getReg();
+        if (Reg == 0)
+          continue;
+        if (O->isDef()) {
+          if (!O->isDead())
+            Defs.push_back(Reg);
+        } else {
+          if (!O->isKill())
             continue;
-          if (O->isDef()) {
-            if (!O->isDead())
-              Defs.push_back(Reg);
-          } else {
-            if (!O->isKill())
-              continue;
-            assert(O->isUse());
-            removeReg(Reg, MCRI);
-          }
-        } else if (O->isRegMask()) {
-          removeRegsInMask(*O, MCRI);
+          assert(O->isUse());
+          removeReg(Reg, MCRI);
         }
-      }
-      // Add defs to the set.
-      for (unsigned i = 0, e = Defs.size(); i != e; ++i) {
-        addReg(Defs[i], MCRI);
+      } else if (O->isRegMask()) {
+        removeRegsInMask(*O, MCRI);
       }
     }
+    // Add defs to the set.
+    for (unsigned i = 0, e = Defs.size(); i != e; ++i) {
+      addReg(Defs[i], MCRI);
+    }
+  }
 
-    /// Adds all registers in the live-in list of block @p BB.
-    void addLiveIns(const MachineBasicBlock &BB, const MCRegisterInfo &MCRI) {
-      for (MachineBasicBlock::livein_iterator L = BB.livein_begin(),
-           LE = BB.livein_end(); L != LE; ++L) {
-        addReg(*L, MCRI);
-      }
+  /// Adds all registers in the live-in list of block @p BB.
+  void addLiveIns(const MachineBasicBlock &BB, const MCRegisterInfo &MCRI) {
+    for (MachineBasicBlock::livein_iterator L = BB.livein_begin(),
+         LE = BB.livein_end(); L != LE; ++L) {
+      addReg(*L, MCRI);
     }
-  };
+  }
+};
 
-}
+} // namespace llvm
 
 #endif





More information about the llvm-commits mailing list