[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