[PATCH] D25157: [compiler-rt] [cmake] Respect COMPILER_RT_BUILD_* consistently for libs & tests

Michał Górny via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 1 06:18:45 PDT 2016


mgorny created this revision.
mgorny added reviewers: beanz, samsonov.
mgorny added a subscriber: cfe-commits.
Herald added subscribers: dberris, kubabrecka, srhines, danalbert, tberghammer.

Refactor the code to respect COMPILER_RT_BUILD_SANITIZERS and COMPILER_RT_BUILD_XRAY consistently for both libraries and tests. In order to achieve that, factor those variables in when setting COMPILER_RT_* in config-ix.cmake. This makes the COMPILER_RT_* variables suitable for determining both whether to build libraries and whether to run tests.

Update the conditionals in lib/CMakeLists.txt appropriately to remove unnecessary double-use of COMPILER_RT_BUILD_* variables when COMPILER_RT_HAS_* already controls the particular component.

Fix sanitizer_common tests to respect COMPILER_RT_HAS_* when determining which tools are supported.


https://reviews.llvm.org/D25157

Files:
  cmake/config-ix.cmake
  lib/CMakeLists.txt
  test/sanitizer_common/CMakeLists.txt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D25157.73195.patch
Type: text/x-patch
Size: 6524 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161001/2616674c/attachment.bin>


More information about the cfe-commits mailing list