[llvm] r256785 - [PGO] Simplify string parsing

Xinliang David Li via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 4 14:09:26 PST 2016


Author: davidxl
Date: Mon Jan  4 16:09:26 2016
New Revision: 256785

URL: http://llvm.org/viewvc/llvm-project?rev=256785&view=rev
Log:
[PGO] Simplify string parsing

Patch Suggested by Vedant.

Modified:
    llvm/trunk/lib/ProfileData/InstrProf.cpp

Modified: llvm/trunk/lib/ProfileData/InstrProf.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ProfileData/InstrProf.cpp?rev=256785&r1=256784&r2=256785&view=diff
==============================================================================
--- llvm/trunk/lib/ProfileData/InstrProf.cpp (original)
+++ llvm/trunk/lib/ProfileData/InstrProf.cpp Mon Jan  4 16:09:26 2016
@@ -245,20 +245,10 @@ int readPGOFuncNameStrings(StringRef Nam
       P += UncompressedSize;
     }
     // Now parse the name strings.
-    size_t NameStart = 0;
-    bool isLast = false;
-    do {
-      size_t NameStop = NameStrings.find(' ', NameStart);
-      if (NameStop == StringRef::npos)
-        NameStop = NameStrings.size();
-      if (NameStop >= NameStrings.size() - 1)
-        isLast = true;
-      StringRef Name = NameStrings.substr(NameStart, NameStop - NameStart);
+    SmallVector<StringRef, 0> Names;
+    NameStrings.split(Names, ' ');
+    for (StringRef &Name : Names)
       Symtab.addFuncName(Name);
-      if (isLast)
-        break;
-      NameStart = NameStop + 1;
-    } while (true);
 
     while (P < EndP && *P == 0)
       P++;




More information about the llvm-commits mailing list