[PATCH] D90016: [clangd] NFC: Rely on ADL in llvm::sys::path calls in marshalling

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 23 05:19:24 PDT 2020


kadircet accepted this revision.
kadircet added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang-tools-extra/clangd/index/remote/marshalling/Marshalling.cpp:30
 
+using llvm::sys::path::append;
+using llvm::sys::path::convert_to_slash;
----------------
nit: I would move those into `clang::clangd::remote` (i.e. between `namespace remote {` and `namespace {`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90016



More information about the cfe-commits mailing list