================
Comment at: lib/Format/ContinuationIndenter.cpp:596
@@ -599,2 +595,3 @@
 
+  StringRef FirstLine = Current.TokenText.substr(0, 
Current.FirstNewlineInText);
   unsigned ColumnsUsed =
----------------
We never use FirstNewlineInText or LastNewlineInText for anything other than 
getting the substring and then calculating the CodePointCount. Getting the 
CodePointCount again is a linear operation that we do repeatedly while finding 
the best layout. How about precomputing the CodePointCount for both values 
instead.


http://llvm-reviews.chandlerc.com/D1556
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to