sammccall added a comment. This makes sense but is hard to debug - is there a reason we don't just store (the relevant parts of) the actual compile command? Size?
================ Comment at: clang-tools-extra/clangd/index/Background.cpp:434 SPAN_ATTACH(Tracer, "file", Cmd.Filename); + size_t CmdHash = + llvm::hash_combine(Cmd.Filename, llvm::join(Cmd.CommandLine, ""), ---------------- only CommandLine and Directory are relevant. Heuristic and Output don't affect anything, and Filename is already part of the storage key. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64018/new/ https://reviews.llvm.org/D64018 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
