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

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 3 13:26:56 PDT 2017


eugenis accepted this revision.
eugenis added inline comments.
This revision is now accepted and ready to land.


================
Comment at: test/cfi/CMakeLists.txt:39
 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)
----------------
replace with "if (CFI_SUPPORTED_ARCH)" so that it is not limited to Linux.



Repository:
  rL LLVM

https://reviews.llvm.org/D35101





More information about the llvm-commits mailing list