[PATCH] D148384: [Demangle] replace use of llvm::StringView w/ std::string_view

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 17 10:24:00 PDT 2023


nickdesaulniers added inline comments.


================
Comment at: llvm/include/llvm/Demangle/ItaniumDemangle.h:19
 
+#include <llvm/ADT/StringViewExtras.h>
+
----------------
MaskRay wrote:
> chapuni wrote:
> > Odd layering...
> Good catch. LLVMDemangle cannot depend on other LLVM libraries.
If these are inline function definitions, then there is no linkage between the libraries?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148384



More information about the llvm-commits mailing list