[PATCH] D61730: [cmake] Remove MSVC C4800 override

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 9 13:35:35 PDT 2019


rnk added a comment.

lgtm

I confirmed that this hasn't been in the default warning set for 2017, which these days I guess is the oldest we support. Feel free to remove other off-by-default warnings, I don't think we need to proactively defend against users adding them to their own cflags, or using old msvc versions.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61730/new/

https://reviews.llvm.org/D61730





More information about the llvm-commits mailing list