[PATCH] D121812: [clang][deps] NFC: De-duplicate clang-cl tests
Sylvain Audi via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 22 13:38:59 PDT 2022
saudi accepted this revision.
saudi added a comment.
This revision is now accepted and ready to land.
It is indeed much cleaner this way`, thanks!
The tests are not anymore the same for `clang-cl` as for `clang`, but I agree what needs testing is the clang-cl specific behaviors, not having the same tests redone as a `clang-cl` flavor.
================
Comment at: clang/test/ClangScanDeps/cl-resource-dir.c:3
+
+// REQUIRES: shell
+
----------------
I was wondering whether it could be a concern that this test will be skipped on Windows systems, where `clang-cl` specific development would most likely occur.
However I'm pretty sure it's ok, since this test is pretty simple and small, has little chances to break during most development iterations, and we would run the tests under linux at some point anyway.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121812/new/
https://reviews.llvm.org/D121812
More information about the cfe-commits
mailing list