Subzero: Fix the name mangling code's base-36 increment.

src/IceGlobalContext.cpp

 //===- subzero/src/IceGlobalContext.cpp - Global context defs ---*- C++ -*-===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 //
 // This file defines aspects of the compilation that persist across
@@ skipping 148 matching lines @@
         }
       }
       if (Found) {
         NewName[NewPos++] = OldName[OldPos++]; // 'S'
         size_t Length = Last - OldPos;
         // NewPos and OldPos point just past the 'S'.
         assert(NewName[NewPos - 1] == 'S');
         assert(OldName[OldPos - 1] == 'S');
         assert(OldName[OldPos + Length] == '_');
         if (AllZs) {
-          // Replace N 'Z' characters with N+1 '0' characters.  (This
-          // is also true for N=0, i.e. S_ ==> S0_ .)
-          for (size_t i = 0; i < Length + 1; ++i) {
+          // Replace N 'Z' characters with a '0' (if N=0) or '1' (if
+          // N>0) followed by N '0' characters.
+          NewName[NewPos++] = (Length ? '1' : '0');
+          for (size_t i = 0; i < Length; ++i) {
             NewName[NewPos++] = '0';
           }
         } else {
           // Iterate right-to-left and increment the base-36 number.
           bool Carry = true;
           for (size_t i = 0; i < Length; ++i) {
             size_t Offset = Length - 1 - i;
             char Ch = OldName[OldPos + Offset];
             if (Carry) {
               Carry = false;
@@ skipping 192 matching lines @@
 
 void Timer::printElapsedUs(GlobalContext *Ctx, const IceString &Tag) const {
   if (Ctx->isVerbose(IceV_Timing)) {
     // Prefixing with '#' allows timing strings to be included
     // without error in textual assembly output.
     Ctx->getStrDump() << "# " << getElapsedUs() << " usec " << Tag << "\n";
   }
 }
 
 } // end of namespace Ice