[PATCH] D73457: [Clang] Warn about 'z' printf modifier in old MSVC.

[Simon Tatham via Phabricator via cfe-commits]

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