[Lldb-commits] [PATCH] D149914: [lldb] Refactor ObjCLanguage::MethodName

[Adrian Prantl via Phabricator via lldb-commits]

aprantl added inline comments.


================
Comment at: lldb/source/Plugins/Language/ObjC/ObjCLanguage.cpp:155
+  llvm::StringRef selector_name = GetSelector();
+  std::string name_sans_category;
+
----------------
bulbazord wrote:
> aprantl wrote:
> > Using an llvm string stream to construct the string might be faster here.
> I tested this and found that `llvm::SmallString<128>` performs about as well as `std::string` on average. `llvm::SmallString<64>` was on average slower though.
> 
> I think I'm going to land this with std::string, we can revisit later if the difference actually is measurable.
Maybe we were talking past each other: I was thinking about https://www.llvm.org/doxygen/classllvm_1_1raw__string__ostream.html


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149914/new/

https://reviews.llvm.org/D149914