[llvm-commits] [llvm] r81699 - /llvm/trunk/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp

[Chris Lattner]

Author: lattner
Date: Sun Sep 13 14:10:08 2009
New Revision: 81699

URL: http://llvm.org/viewvc/llvm-project?rev=81699&view=rev
Log:
reduce indentation with early exit.

Modified:
    llvm/trunk/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp

Modified: llvm/trunk/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp?rev=81699&r1=81698&r2=81699&view=diff

==============================================================================
--- llvm/trunk/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp (original)
+++ llvm/trunk/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp Sun Sep 13 14:10:08 2009
@@ -575,62 +575,62 @@
 
 void X86IntelAsmPrinter::EmitString(const ConstantArray *CVA) const {
   unsigned NumElts = CVA->getNumOperands();
-  if (NumElts) {
-    // ML does not have escape sequences except '' for '.  It also has a maximum
-    // string length of 255.
-    unsigned len = 0;
-    bool inString = false;
-    for (unsigned i = 0; i < NumElts; i++) {
-      int n = cast<ConstantInt>(CVA->getOperand(i))->getZExtValue() & 255;
-      if (len == 0)
-        O << "\tdb ";
+  if (NumElts == 0) return;
+  
+  // ML does not have escape sequences except '' for '.  It also has a maximum
+  // string length of 255.
+  unsigned len = 0;
+  bool inString = false;
+  for (unsigned i = 0; i < NumElts; i++) {
+    int n = cast<ConstantInt>(CVA->getOperand(i))->getZExtValue() & 255;
+    if (len == 0)
+      O << "\tdb ";
 
-      if (n >= 32 && n <= 127) {
-        if (!inString) {
-          if (len > 0) {
-            O << ",'";
-            len += 2;
-          } else {
-            O << "'";
-            len++;
-          }
-          inString = true;
-        }
-        if (n == '\'') {
-          O << "'";
-          len++;
-        }
-        O << char(n);
-      } else {
-        if (inString) {
-          O << "'";
-          len++;
-          inString = false;
-        }
+    if (n >= 32 && n <= 127) {
+      if (!inString) {
         if (len > 0) {
-          O << ",";
+          O << ",'";
+          len += 2;
+        } else {
+          O << "'";
           len++;
         }
-        O << n;
-        len += 1 + (n > 9) + (n > 99);
+        inString = true;
       }
-
-      if (len > 60) {
-        if (inString) {
-          O << "'";
-          inString = false;
-        }
-        O << "\n";
-        len = 0;
+      if (n == '\'') {
+        O << "'";
+        len++;
+      }
+      O << char(n);
+    } else {
+      if (inString) {
+        O << "'";
+        len++;
+        inString = false;
       }
+      if (len > 0) {
+        O << ",";
+        len++;
+      }
+      O << n;
+      len += 1 + (n > 9) + (n > 99);
     }
 
-    if (len > 0) {
-      if (inString)
+    if (len > 60) {
+      if (inString) {
         O << "'";
+        inString = false;
+      }
       O << "\n";
+      len = 0;
     }
   }
+
+  if (len > 0) {
+    if (inString)
+      O << "'";
+    O << "\n";
+  }
 }
 
 // Include the auto-generated portion of the assembly writer.