Use early exit to reduce indentation.
authorBob Wilson <bob.wilson@apple.com>
Mon, 12 Oct 2009 19:04:03 +0000 (19:04 +0000)
committerBob Wilson <bob.wilson@apple.com>
Mon, 12 Oct 2009 19:04:03 +0000 (19:04 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@83874 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/ARM/ARMConstantIslandPass.cpp

index ed3c32532ab6c7e4fdd3e18a1474d41851ca6b3c..955c9c152fc3d1402eed86c2e95732643164c58d 100644 (file)
@@ -956,28 +956,29 @@ bool ARMConstantIslands::LookForWater(CPUser &U, unsigned UserOffset,
                                       MachineBasicBlock *&NewMBB) {
   water_iterator IPThatWouldPad;
   MachineBasicBlock* WaterBBThatWouldPad = NULL;
-  if (!WaterList.empty()) {
-    for (water_iterator IP = prior(WaterList.end()),
-           B = WaterList.begin();; --IP) {
-      MachineBasicBlock* WaterBB = *IP;
-      if (WaterIsInRange(UserOffset, WaterBB, U)) {
-        unsigned WBBId = WaterBB->getNumber();
-        if (isThumb &&
-            (BBOffsets[WBBId] + BBSizes[WBBId])%4 != 0) {
-          // This is valid Water, but would introduce padding.  Remember
-          // it in case we don't find any Water that doesn't do this.
-          if (!WaterBBThatWouldPad) {
-            WaterBBThatWouldPad = WaterBB;
-            IPThatWouldPad = IP;
-          }
-        } else {
-          NewMBB = AcceptWater(WaterBB, IP);
-          return true;
+  if (WaterList.empty())
+    return false;
+
+  for (water_iterator IP = prior(WaterList.end()),
+         B = WaterList.begin();; --IP) {
+    MachineBasicBlock* WaterBB = *IP;
+    if (WaterIsInRange(UserOffset, WaterBB, U)) {
+      unsigned WBBId = WaterBB->getNumber();
+      if (isThumb &&
+          (BBOffsets[WBBId] + BBSizes[WBBId])%4 != 0) {
+        // This is valid Water, but would introduce padding.  Remember
+        // it in case we don't find any Water that doesn't do this.
+        if (!WaterBBThatWouldPad) {
+          WaterBBThatWouldPad = WaterBB;
+          IPThatWouldPad = IP;
         }
+      } else {
+        NewMBB = AcceptWater(WaterBB, IP);
+        return true;
       }
-      if (IP == B)
-        break;
     }
+    if (IP == B)
+      break;
   }
   if (isThumb && WaterBBThatWouldPad) {
     NewMBB = AcceptWater(WaterBBThatWouldPad, IPThatWouldPad);