[opt-viewer] Allow users to set LLVM_HAVE_OPT_VIEWER_MODULES manually from the CMake

Check right before setting LLVM_HAVE_OPT_VIEWER_MODULES
so that if the user specifies the variable on the commandline
it takes precedence. If not, fall back to the old logic

Upstream-Status: Submitted [llvm#151845]
Signed-off-by: Khem Raj <[email protected]>

Author: kraj

llvm/CMakeLists.txt

@@ -7,6 +7,8 @@ set(LLVM_COMMON_CMAKE_UTILS ${CMAKE_CURRENT_SOURCE_DIR}/../cmake)
 include(${LLVM_COMMON_CMAKE_UTILS}/Modules/CMakePolicy.cmake
   NO_POLICY_SCOPE)
 
+option(LLVM_HAVE_OPT_VIEWER_MODULES "Enable opt-viewer modules support" ON)
+
 # Builds with custom install names and installation rpath setups may not work
 # in the build tree. Allow these cases to use CMake's default build tree
 # behavior by setting `LLVM_NO_INSTALL_NAME_DIR_FOR_BUILD_TREE` to do this.

llvm/cmake/config-ix.cmake

@@ -805,10 +805,17 @@ foreach(module ${PYTHON_MODULES})
   find_python_module(${module})
 endforeach()
 
-if(PY_PYGMENTS_FOUND AND PY_PYGMENTS_LEXERS_C_CPP_FOUND AND PY_YAML_FOUND)
-  set (LLVM_HAVE_OPT_VIEWER_MODULES 1)
+if(NOT DEFINED LLVM_HAVE_OPT_VIEWER_MODULES)
+  if(PY_PYGMENTS_FOUND AND PY_PYGMENTS_LEXERS_C_CPP_FOUND AND PY_YAML_FOUND)
+    set (LLVM_HAVE_OPT_VIEWER_MODULES 1)
+  else()
+    set (LLVM_HAVE_OPT_VIEWER_MODULES 0)
+  endif()
+elseif(LLVM_HAVE_OPT_VIEWER_MODULES)
+  # Use the user-provided value
+  set(LLVM_HAVE_OPT_VIEWER_MODULES 1)
 else()
-  set (LLVM_HAVE_OPT_VIEWER_MODULES 0)
+  set(LLVM_HAVE_OPT_VIEWER_MODULES 0)
 endif()
 
 function(llvm_get_host_prefixes_and_suffixes)

llvm/include/llvm/Config/config.h.cmake

@@ -287,6 +287,9 @@
 /* Define to a function implementing strdup */
 #cmakedefine strdup ${strdup}
 
+/* Define indicating opt-viewer modules */
+#cmakedefine LLVM_HAVE_OPT_VIEWER_MODULES
+
 /* Whether GlobalISel rule coverage is being collected */
 #cmakedefine01 LLVM_GISEL_COV_ENABLED