rsmith added a comment.

In D74051#1872856 <https://reviews.llvm.org/D74051#1872856>, @MaskRay wrote:

> @rsmith Does `clang/test/utils/` (a new directory) look appropriate to you?


That seems fine to me. Following that pattern, I think we should also move:

- `test/clang-rename` -> `test/tools/clang-rename`
- `test/ClangScanDeps` -> `test/tools/clang-scan-deps`
- `test/TableGen` -> `test/utils/TableGen`

(We might want to move part of `test/Index` to `test/tools/libclang` and parts 
to `test/tools/c-index-test`, but we'd need to decide for each test whether 
it's a test for Clang's behavior, a test for libclang's wrapping behavior, or a 
test for the c-index-test tool itself. Let's leave that alone for now.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74051



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

Reply via email to