nridge added a comment. In D56370#1426234 <https://reviews.llvm.org/D56370#1426234>, @kadircet wrote:
> Unfortunately we usually test LSP layer with lit tests, and since we don't > have any lit tests for this use-case it wasn't caught. Could you add a lit > test for overall use case? You can see examples inside > clang-tools-extra/test/clangd/ folder. Are these tests written manually, or with the help or some kind of tool? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56370/new/ https://reviews.llvm.org/D56370 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
