[PATCH] D35101: [cfi] only add cfi tests on supported arches

[Sean Fertile via Phabricator via llvm-commits]

sfertile updated this revision to Diff 109596.
sfertile added a comment.

Removed unnecessary loop.


Repository:
  rL LLVM

https://reviews.llvm.org/D35101

Files:
  test/cfi/CMakeLists.txt


Index: test/cfi/CMakeLists.txt
===================================================================
--- test/cfi/CMakeLists.txt
+++ test/cfi/CMakeLists.txt
@@ -36,11 +36,13 @@
   # FIXME: enable ThinLTO tests after fixing http://llvm.org/pr32770
   add_cfi_test_suites(True False)
 else()
-  add_cfi_test_suites(False False)
-  add_cfi_test_suites(False True)
-  if (COMPILER_RT_HAS_LLD)
-    add_cfi_test_suites(True False)
-    add_cfi_test_suites(True True)
+  if (COMPILER_RT_HAS_CFI)
+    add_cfi_test_suites(False False)
+    add_cfi_test_suites(False True)
+    if (COMPILER_RT_HAS_LLD)
+      add_cfi_test_suites(True False)
+      add_cfi_test_suites(True True)
+    endif()
   endif()
 endif()
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35101.109596.patch
Type: text/x-patch
Size: 705 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170803/f0972644/attachment.bin>