[PATCH] D40204: [test-suite] Fix Xcode SDK cmake cache for benchmarks using re_comp

Francis Visoiu Mistrih via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 17 18:19:00 PST 2017


This revision was automatically updated to reflect the committed changes.
Closed by commit rL318585: [test-suite] Fix Xcode SDK cmake cache for benchmarks using re_comp (authored by thegameg).

Changed prior to commit:
  https://reviews.llvm.org/D40204?vs=123456&id=123457#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D40204

Files:
  test-suite/trunk/cmake/caches/util/xcode_sdk.cmake


Index: test-suite/trunk/cmake/caches/util/xcode_sdk.cmake
===================================================================
--- test-suite/trunk/cmake/caches/util/xcode_sdk.cmake
+++ test-suite/trunk/cmake/caches/util/xcode_sdk.cmake
@@ -44,6 +44,13 @@
   # required to pick up the correct linker to the early compiler tests. However
   # it does have an option to not link in the early tests.
   set(CMAKE_TRY_COMPILE_TARGET_TYPE STATIC_LIBRARY CACHE STRING "")
+
+  # Some benchmarks check for the function re_comp by using
+  # check_function_exists. This function is not available in any Xcode SDK, and
+  # the check returns YES with custom built compilers when we cross-compile,
+  # because of "CMAKE_TRY_COMPILE_TARGET_TYPE STATIC_LIBRARY" explained just
+  # above. We work around that by forcing it to NO.
+  set(HAVE_RE_COMP CACHE BOOL NO)
 else()
   # Search and use compiler coming with the SDK.
   # Note that we do not search CMAKE_CXX_COMPILER here. cmake will try


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D40204.123457.patch
Type: text/x-patch
Size: 983 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171118/7af65041/attachment.bin>


More information about the llvm-commits mailing list