[PATCH] D57625: [TargetLibraryInfo] Update run time support for Windows

[Evandro Menezes via Phabricator via llvm-commits]

evandro marked 2 inline comments as done.
evandro added inline comments.


================
Comment at: llvm/lib/Analysis/TargetLibraryInfo.cpp:165
+                         T.getArch() == Triple::arm ||
+                         T.getArch() == Triple::x86_64);
+
----------------
rnk wrote:
> I think you can use `T.getEnvironmentVersion()` here to get the MSVC version. Maybe make it conditional on `isWindowsMSVCEnvironment()`. If there is no version or it isn't an MSVC environment, just assume full C99 support is available (i.e. the UCRT is in use).
What's the correlation between MSVCRT major version and VS release year?


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

https://reviews.llvm.org/D57625