[PATCH] D120115: [clangd] Tweak --query-driver to ignore slash direction on windows

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 18 09:06:20 PST 2022


sammccall marked an inline comment as done.
sammccall added inline comments.


================
Comment at: clang-tools-extra/clangd/QueryDriverDatabase.cpp:278
+    } else if (llvm::sys::path::is_separator(Glob[I]) &&
+               llvm::sys::path::is_separator('/') &&
+               llvm::sys::path::is_separator('\\')) {
----------------
kadircet wrote:
> nit: maybe extract `llvm::sys::path::is_separator('/') && llvm::sys::path::is_separator('\\')` into a `bool PlatformIsSlashInsensitive` ?
We require the separators to be exactly \ and / for the regex to be correct, so I'd rather keep the literals here.

(Call me paranoid, but I grew up on a system with `:` for separators...)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120115



More information about the cfe-commits mailing list