[PATCH] D61187: [clangd] Move clangd tests to clangd directory. check-clangd is no longer part of check-clang-tools.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 1 23:50:41 PDT 2019


sammccall added a comment.

So, apologies everyone for the trouble caused by this patch, and thanks for all the information.
It was a mistake to try to change so many things at once. I'll switch the layout back as far as possible while preserving the directory structure.

This is a frustrating as the complexity here has been causing real problems. For clangd, the costs seem to outweigh the benefits (such as re-running lit tests and supporting visual studio project generation). But I accept that this is something that each project doesn't get to decide, and I'll need to accept these decisions or push to move clangd out of LLVM.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61187





More information about the cfe-commits mailing list