[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