[compiler-rt] r290439 - [scudo] Use the macro get_test_cc_for_arch for the tests

Kostya Kortchinsky via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 23 10:40:47 PST 2016


Author: cryptoad
Date: Fri Dec 23 12:40:47 2016
New Revision: 290439

URL: http://llvm.org/viewvc/llvm-project?rev=290439&view=rev
Log:
[scudo] Use the macro get_test_cc_for_arch for the tests

Summary: The macro was introduced with D26929, use it in Scudo as well.

Reviewers: kcc, alekseyshl, kubabrecka

Subscribers: llvm-commits, danalbert, srhines, mgorny

Differential Revision: https://reviews.llvm.org/D28066

Modified:
    compiler-rt/trunk/test/scudo/CMakeLists.txt

Modified: compiler-rt/trunk/test/scudo/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/scudo/CMakeLists.txt?rev=290439&r1=290438&r2=290439&view=diff
==============================================================================
--- compiler-rt/trunk/test/scudo/CMakeLists.txt (original)
+++ compiler-rt/trunk/test/scudo/CMakeLists.txt Fri Dec 23 12:40:47 2016
@@ -17,16 +17,7 @@ set(SCUDO_TEST_ARCH ${SCUDO_SUPPORTED_AR
 foreach(arch ${SCUDO_TEST_ARCH})
   set(SCUDO_TEST_TARGET_ARCH ${arch})
   string(TOLOWER "-${arch}" SCUDO_TEST_CONFIG_SUFFIX)
-  
-  if(ANDROID OR ${arch} MATCHES "arm|aarch64")
-    # This is only true if we are cross-compiling.
-    # Build all tests with host compiler and use host tools.
-    set(SCUDO_TEST_TARGET_CFLAGS ${COMPILER_RT_TEST_COMPILER_CFLAGS})
-  else()
-    get_target_flags_for_arch(${arch} SCUDO_TEST_TARGET_CFLAGS)
-    string(REPLACE ";" " " SCUDO_TEST_TARGET_CFLAGS "${SCUDO_TEST_TARGET_CFLAGS}")
-  endif()
-
+  get_test_cc_for_arch(${arch} SCUDO_TEST_TARGET_CC SCUDO_TEST_TARGET_CFLAGS)
   string(TOUPPER ${arch} ARCH_UPPER_CASE)
   set(CONFIG_NAME ${ARCH_UPPER_CASE}${OS_NAME}Config)
 




More information about the llvm-commits mailing list