[clang-tools-extra] [clangd] Add background index path remapping tool (PR #185482)

Justin Cady via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 12 06:03:04 PDT 2026


justincady wrote:

> * We don't seem to support Windows paths. This is probably acceptable at this stage, but it needs to be explicitly written into the code as a FIXME. Right now, this can only be determined by testing.

Thanks. Updated to explicitly note this.

> * We use both llvm::errs() and clangd Logger (elog) for logging. I think the clangd logger (log(), vlog(), etc.) should be used everywhere.

Agreed. Fixed as suggested.

> * It's not entirely clear why we depend on zlib in the test. The index can be either packed or unpacked.

Thanks for catching this. Removed.

https://github.com/llvm/llvm-project/pull/185482


More information about the cfe-commits mailing list