[PATCH] Adding SmallString support to Twine
Paweł Bylica
chfast at gmail.com
Mon Mar 16 01:59:03 PDT 2015
================
Comment at: tools/clang/lib/Sema/CodeCompleteConsumer.cpp:254
@@ -253,2 +253,3 @@
-const char *CodeCompletionAllocator::CopyString(StringRef String) {
+const char *CodeCompletionAllocator::CopyString(const Twine &T) {
+ SmallString<128> Data;
----------------
I liked the argument name `String` better than `T`. I know the type has changed but `CopyString` copies strings.
http://reviews.llvm.org/D6372
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list