[CMake] Temporarily remove the LLVM_ENABLE_IDE option
authorChris Bieneman <chris.bieneman@me.com>
Thu, 11 Oct 2018 04:06:14 +0000 (04:06 +0000)
committerChris Bieneman <chris.bieneman@me.com>
Thu, 11 Oct 2018 04:06:14 +0000 (04:06 +0000)
All uses of this option have been removed, and the intent is to change the purpose and default value of this option. To prevent it from having impacts on users, this patch temporarily removes the option and purges it from CMake caches. In a few days, once this has propagated to contributors I will re-introduce the option with the new default value.

llvm-svn: 344219

llvm/cmake/modules/HandleLLVMOptions.cmake

index 0daaf7d..85aebf6 100644 (file)
@@ -868,12 +868,16 @@ else()
   set(LLVM_ENABLE_PLUGINS ON)
 endif()
 
-set(LLVM_ENABLE_IDE_default OFF)
-if (XCODE OR MSVC_IDE OR CMAKE_EXTRA_GENERATOR)
-  set(LLVM_ENABLE_IDE_default ON)
-endif()
-option(LLVM_ENABLE_IDE "Generate targets and process sources for use with an IDE"
-    ${LLVM_ENABLE_IDE_default})
+# Remove LLVM_ENABLE_IDE from the CMake cache. This is a temporary change to
+# allow CMake caches to be cleaned up so that we can change the default for this
+# option and how it is used.
+unset(LLVM_ENABLE_IDE CACHE)
+#set(LLVM_ENABLE_IDE_default OFF)
+#if (XCODE OR MSVC_IDE OR CMAKE_EXTRA_GENERATOR)
+#  set(LLVM_ENABLE_IDE_default ON)
+#endif()
+#option(LLVM_ENABLE_IDE "Generate targets and process sources for use with an IDE"
+#    ${LLVM_ENABLE_IDE_default})
 
 function(get_compile_definitions)
   get_directory_property(top_dir_definitions DIRECTORY ${CMAKE_SOURCE_DIR} COMPILE_DEFINITIONS)