[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:07:25 PST 2017


thegameg created this revision.
Herald added a subscriber: mgorny.

cmake thinks it found re_comp after r317058 <https://reviews.llvm.org/rL317058>.

Test affected:

- MultiSource/Benchmarks/Prolangs-C/plot2fig
- MultiSource/Benchmarks/Prolangs-C/archie-client


https://reviews.llvm.org/D40204

Files:
  cmake/caches/util/xcode_sdk.cmake


Index: cmake/caches/util/xcode_sdk.cmake
===================================================================
--- cmake/caches/util/xcode_sdk.cmake
+++ 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.123456.patch
Type: text/x-patch
Size: 932 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171118/eaa6bd54/attachment.bin>


More information about the llvm-commits mailing list