[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