thakis added a comment.

This breaks tests on Windows: http://45.33.8.238/win/6812/step_7.txt
Since we auto-detect -fmsc-version if it's not explicitly installed and since 
this warning is on by default, it makes the test suite depend on the 
environment a good bit. Given how old 2015 is by now, I'm not sure this 
complexity and subtlety is worth the benefit?

In any case, I'll revert this for now to heal the bots.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73457



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to