[PATCH] D50875: Move lib/Demangle under lib/Support. No functionality change intended.
Richard Smith - zygoloid via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 16 16:13:04 PDT 2018
rsmith added a comment.
I removed the copy of Compiler.h from lib/Support/Demangle, because there is no longer any layering reason to not just use the version from include/llvm/Support. (libc++abi will still need its own Compiler.h.)
I did not remove the StringView.h from lib/Support/Demangle, because the `StringView` class has a different interface from both `llvm::StringRef` and `std::string_view`, and keeping that file around to provide the interface that the demangling code wants seems like the best path for now. (libc++abi presumably can't depend on `std::string_view`, so needs its own implementation anyway.)
Repository:
rL LLVM
https://reviews.llvm.org/D50875
More information about the llvm-commits
mailing list