ilya-biryukov requested changes to this revision.
ilya-biryukov added a subscriber: sammccall.
ilya-biryukov added a comment.
This revision now requires changes to proceed.

Open question: I also feel like the best option here is to fix the tests, but 
I'm not sure how hard that would be. @sammccall any thoughts?
I suspect the particular tests are flaky is because they rely on timeouts, not 
sure it's easy to disentangle them. Therefore, some workaround seems reasonable

If we land this, we should ensure that when `CLANG_INCLUDE_TESTS` is off, 
Clangd tests don't run.
Clangd should respect the LLVM-wide options even if it has a more specific one 
for the project itself.

I'm marking as requiring changes mostly for the latter comment about 
`CLANG_INCLUDE_TESTS`.


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

https://reviews.llvm.org/D158566

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to