[PATCH] D15179: [sanitizer] Cache results of darwin_test_archs

[Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL254618: [sanitizer] Cache results of darwin_test_archs (authored by kuba.brecka).

Changed prior to commit:
  http://reviews.llvm.org/D15179?vs=41725&id=41764#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D15179

Files:
  compiler-rt/trunk/cmake/Modules/CompilerRTDarwinUtils.cmake

Index: compiler-rt/trunk/cmake/Modules/CompilerRTDarwinUtils.cmake
===================================================================
--- compiler-rt/trunk/cmake/Modules/CompilerRTDarwinUtils.cmake
+++ compiler-rt/trunk/cmake/Modules/CompilerRTDarwinUtils.cmake
@@ -46,6 +46,11 @@
 # This function takes an OS and a list of architectures and identifies the
 # subset of the architectures list that the installed toolchain can target.
 function(darwin_test_archs os valid_archs)
+  if(${valid_archs})
+    message(STATUS "Using cached valid architectures for ${os}.")
+    return()
+  endif()
+
   set(archs ${ARGN})
   message(STATUS "Finding valid architectures for ${os}...")
   set(SIMPLE_CPP ${CMAKE_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/src.cpp)
@@ -75,7 +80,8 @@
       list(APPEND working_archs ${arch})
     endif()
   endforeach()
-  set(${valid_archs} ${working_archs} PARENT_SCOPE)
+  set(${valid_archs} ${working_archs}
+    CACHE STRING "List of valid architectures for platform ${os}.")
 endfunction()
 
 # This function checks the host cpusubtype to see if it is post-haswell. Haswell


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15179.41764.patch
Type: text/x-patch
Size: 1099 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151203/8a62e226/attachment.bin>