[PATCH] D74051: Move update_cc_test_checks.py tests to clang

Richard Smith - zygoloid via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 14 02:21:49 PST 2020


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





More information about the llvm-commits mailing list