[cfe-commits] r49767 - /cfe/trunk/lib/Rewrite/HTMLRewrite.cpp

Chris Lattner sabre at nondot.org
Tue Apr 15 21:11:35 PDT 2008


Author: lattner
Date: Tue Apr 15 23:11:35 2008
New Revision: 49767

URL: http://llvm.org/viewvc/llvm-project?rev=49767&view=rev
Log:
switch AddLineNumber to use a SmallString instead of sstream.  This
speeds up -emit-html on ted's testcase by 29% (.138 -> 0.107s) in a
release build.


Modified:
    cfe/trunk/lib/Rewrite/HTMLRewrite.cpp

Modified: cfe/trunk/lib/Rewrite/HTMLRewrite.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Rewrite/HTMLRewrite.cpp?rev=49767&r1=49766&r2=49767&view=diff

==============================================================================
--- cfe/trunk/lib/Rewrite/HTMLRewrite.cpp (original)
+++ cfe/trunk/lib/Rewrite/HTMLRewrite.cpp Tue Apr 15 23:11:35 2008
@@ -15,6 +15,7 @@
 #include "clang/Rewrite/Rewriter.h"
 #include "clang/Rewrite/HTMLRewrite.h"
 #include "clang/Basic/SourceManager.h"
+#include "llvm/ADT/SmallString.h"
 #include "llvm/Support/MemoryBuffer.h"
 #include <sstream>
 
@@ -113,17 +114,18 @@
 
 static void AddLineNumber(Rewriter& R, unsigned LineNo,
                           SourceLocation B, SourceLocation E) {
-    
-  std::ostringstream os;
-  os << "<tr><td class=\"num\" id=\"LN" << LineNo << "\">" 
-     << LineNo << "</td><td class=\"line\">";
-
+  llvm::SmallString<100> Str;
+  Str += "<tr><td class=\"num\" id=\"LN";
+  Str.append_uint(LineNo);
+  Str += "\">";
+  Str.append_uint(LineNo);
+  Str += "</td><td class=\"line\">";
+  
   if (B == E) { // Handle empty lines.
-    os << " </td></tr>";
-    R.InsertStrBefore(B, os.str());
-  }
-  else {
-    R.InsertStrBefore(B, os.str());
+    Str += " </td></tr>";
+    R.InsertTextBefore(B, &Str[0], Str.size());
+  } else {
+    R.InsertTextBefore(B, &Str[0], Str.size());
     R.InsertCStrBefore(E, "</td></tr>");
   }
 }





More information about the cfe-commits mailing list