Skip to content

Commit

Permalink
Remove obsolete msvc workaround from AddLLVM.cmake
Browse files Browse the repository at this point in the history
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 <atrick@apple.com>
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
  • Loading branch information
sstamenova committed Sep 9, 2021
1 parent e69d402 commit 584a5d1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion llvm/cmake/modules/AddLLVM.cmake
Expand Up @@ -1638,7 +1638,7 @@ function(configure_lit_site_cfg site_in site_out)
set(ENABLE_SHARED "0")
endif()

if(LLVM_ENABLE_ASSERTIONS AND NOT MSVC_IDE)
if(LLVM_ENABLE_ASSERTIONS)
set(ENABLE_ASSERTIONS "1")
else()
set(ENABLE_ASSERTIONS "0")
Expand Down

0 comments on commit 584a5d1

Please sign in to comment.