[llvm-branch-commits] [compiler-rt] Cleanup use of COMPILER_RT_INCLUDE_TESTS (PR #98246)
Fangrui Song via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Tue Jul 9 18:39:05 PDT 2024
https://github.com/MaskRay approved this pull request.
I am not familiar with how the tests cmake work, but this patch indeed improves consistency.
For example, `foreach(arch ${MEMPROF_TEST_ARCH})` is at the toplevel while its unittest `foreach` is guarded by `COMPILER_RT_INCLUDE_TESTS`. Removing `COMPILER_RT_INCLUDE_TESTS` is great.
https://github.com/llvm/llvm-project/pull/98246
More information about the llvm-branch-commits
mailing list