[PATCH] D148181: [Demangle] make llvm::demangle take a StringRef; NFC

Sam Clegg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 12 16:08:16 PDT 2023


sbc100 added inline comments.


================
Comment at: llvm/lib/Demangle/Demangle.cpp:29
 
-std::string llvm::demangle(const std::string &MangledName) {
+std::string llvm::demangle(const StringRef MangledName) {
   std::string Result;
----------------
Should `const` be removed?  (i.e. is StringRef not inherently const?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148181



More information about the cfe-commits mailing list