[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