[PATCH] Adding SmallString support to Twine

Paweł Bylica chfast at gmail.com
Fri Mar 13 17:21:26 PDT 2015


================
Comment at: include/llvm/MC/MCContext.h:236-238
@@ -235,4 +235,5 @@
     /// @param Name - The symbol name, which must be unique across all symbols.
     MCSymbol *GetOrCreateSymbol(StringRef Name);
+    MCSymbol *GetOrCreateSymbol(const SmallVectorImpl<char> &Name);
     MCSymbol *GetOrCreateSymbol(const Twine &Name);
 
----------------
If Twine can handle SmallString now, wouldn't it be better to reduce the number of overloads in this case?

http://reviews.llvm.org/D6372

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list