simon_tatham added a comment.

I wanted this to go in because I'm actually using it – pre-2015 C libraries are 
useful to link against if you need an application to run on very old versions 
of Windows, and that means you need the compiler to warn you if you do 
something those libraries don't support.

I see that auto-detecting the default MS compatibility version is not very 
compatible with keeping the test suite independent of the environment, but 
surely that's a general problem with the autodetection policy, nothing to do 
with this particular patch?

Would you be happy with just removing the RUN line in this test that doesn't 
specify a default compatibility version, so that every remaining RUN line 
specifies an explicit default?


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