[PATCH] D123777: [CMake] Check for problematic MSVC + /arch:AVX configuration

[Tobias Hieta via Phabricator via llvm-commits]

thieta created this revision.
thieta added reviewers: RKSimon, dexonsmith, mehdi_amini.
Herald added a subscriber: mgorny.
Herald added a project: All.
thieta requested review of this revision.
Herald added a project: LLVM.

Add a new CMake file to expand on for more problematic configurations
in the future.

Related to #54645


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D123777

Files:
  llvm/cmake/config-ix.cmake
  llvm/cmake/modules/CheckProblematicConfigurations.cmake
  llvm/cmake/modules/HandleLLVMOptions.cmake


Index: llvm/cmake/modules/HandleLLVMOptions.cmake
===================================================================
--- llvm/cmake/modules/HandleLLVMOptions.cmake
+++ llvm/cmake/modules/HandleLLVMOptions.cmake
@@ -7,6 +7,7 @@
 string(TOUPPER "${CMAKE_BUILD_TYPE}" uppercase_CMAKE_BUILD_TYPE)
 
 include(CheckCompilerVersion)
+include(CheckProblematicConfigurations)
 include(HandleLLVMStdlib)
 include(CheckCCompilerFlag)
 include(CheckCXXCompilerFlag)
Index: llvm/cmake/modules/CheckProblematicConfigurations.cmake
===================================================================
--- /dev/null
+++ llvm/cmake/modules/CheckProblematicConfigurations.cmake
@@ -0,0 +1,19 @@
+
+option(LLVM_ALLOW_PROBLEMATIC_CONFIGURATIONS OFF "Set this option to ON to force problematic toolchain configurations. Use on your own risk.")
+
+macro(LOG_PROBLEMATIC MESSAGE)
+  if(LLVM_ALLOW_PROBLEMATIC_CONFIGURATIONS)
+    message(WARNING "${MESSAGE}")
+  else()
+    message(FATAL_ERROR "${MESSAGE}\nYou can force usage of this configuration by passing -DLLVM_ALLOW_PROBLEMATIC_CONFIGURATIONS=ON")
+  endif()
+endmacro()
+
+# MSVC and /arch:AVX is untested and have created problems before. See:
+# https://github.com/llvm/llvm-project/issues/54645
+if(${CMAKE_CXX_COMPILER_ID} STREQUAL MSVC)
+  string(TOLOWER "${CMAKE_CXX_FLAGS} ${CMAKE_C_FLAGS}" _FLAGS)
+  if(_FLAGS MATCHES "/arch:avx[0-9]*")
+    LOG_PROBLEMATIC("Compiling LLVM with MSVC and the /arch:AVX flag is known to cause issues with parts of LLVM. Use clang-cl if you want to enable AVX instructions.")
+  endif()
+endif()
Index: llvm/cmake/config-ix.cmake
===================================================================
--- llvm/cmake/config-ix.cmake
+++ llvm/cmake/config-ix.cmake
@@ -12,6 +12,7 @@
 include(CMakePushCheckState)
 
 include(CheckCompilerVersion)
+include(CheckProblematicConfigurations)
 include(HandleLLVMStdlib)
 
 if( UNIX AND NOT (APPLE OR BEOS OR HAIKU) )


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123777.422800.patch
Type: text/x-patch
Size: 1951 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220414/7ab20200/attachment.bin>