[PATCH] D149104: [Demangle] make llvm::demangle take std::string_view rather than const std::string&

[Fangrui Song via Phabricator via cfe-commits]

MaskRay added inline comments.


================
Comment at: lld/ELF/Symbols.cpp:50
+    std::string root = symName.str();
+    return demangle(root);
+  }
----------------
`return demangle(symName.str());`

perhaps `return demangle(symName);` works as well?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149104

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits