Skip to content

Commit 584a5d1

Browse files
committed
Remove obsolete msvc workaround from AddLLVM.cmake
The original change to add the workaround is from 10 years ago and a lot has happened with msvc and cmake and llvm's usage of cmake since and we no longer need the workaround for any scenarios that I am aware of. Build more is now correctly configured for multi-configuration generators such as Visual Studio. The workaround is, however, causing issues with some of the recent mlir tests as because of the workaround we cannot correctly determine whether assertions are enabled (see https://reviews.llvm.org/D105961). The original change is: ``` commit b46fdac Author: Andrew Trick <[email protected]> Date: Tue Jun 28 16:32:01 2011 cmake: Our MSVC build does not support config-time build mode. llvm-svn: 134008 ``` Reviewed By: mehdi_amini Differential Revision: https://reviews.llvm.org/D109521
1 parent e69d402 commit 584a5d1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

llvm/cmake/modules/AddLLVM.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1638,7 +1638,7 @@ function(configure_lit_site_cfg site_in site_out)
16381638
set(ENABLE_SHARED "0")
16391639
endif()
16401640

1641-
if(LLVM_ENABLE_ASSERTIONS AND NOT MSVC_IDE)
1641+
if(LLVM_ENABLE_ASSERTIONS)
16421642
set(ENABLE_ASSERTIONS "1")
16431643
else()
16441644
set(ENABLE_ASSERTIONS "0")

0 commit comments

Comments
 (0)