Skip to content

Commit

Permalink
[cmake] Remove MSVC C4800 override
Browse files Browse the repository at this point in the history
Remove C4800 : ''type' : forcing value to bool 'true' or 'false' (performance warning)' from the list of forced disabled warnings.

I'm not seeing any regressions in VS2017/VS2019 llvm/clang builds from removing this.

Differential Revision: https://reviews.llvm.org/D61730

llvm-svn: 360417
  • Loading branch information
RKSimon authored and MrSidims committed May 17, 2019
1 parent a9d2b39 commit 6a99186
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion llvm/cmake/modules/HandleLLVMOptions.cmake
Expand Up @@ -519,7 +519,6 @@ if (MSVC)
-wd4503 # Suppress ''identifier' : decorated name length exceeded, name was truncated'
-wd4624 # Suppress ''derived class' : destructor could not be generated because a base class destructor is inaccessible'
-wd4722 # Suppress 'function' : destructor never returns, potential memory leak
-wd4800 # Suppress ''type' : forcing value to bool 'true' or 'false' (performance warning)'
-wd4100 # Suppress 'unreferenced formal parameter'
-wd4127 # Suppress 'conditional expression is constant'
-wd4512 # Suppress 'assignment operator could not be generated'
Expand Down

0 comments on commit 6a99186

Please sign in to comment.