Skip to content

Commit

Permalink
[Support] Fix warning: unknown pragma ignored for mingw target
Browse files Browse the repository at this point in the history
Differential Revision: https://reviews.llvm.org/D54133

llvm-svn: 346218
  • Loading branch information
mstorsjo committed Nov 6, 2018
1 parent 4855b74 commit c6fcdd3
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions llvm/lib/Support/Windows/WindowsSupport.h
Expand Up @@ -92,15 +92,19 @@ inline llvm::VersionTuple GetWindowsOSVersion() {
OSVERSIONINFOEX info;
ZeroMemory(&info, sizeof(OSVERSIONINFOEX));
info.dwOSVersionInfoSize = sizeof(OSVERSIONINFOEX);
#if defined(_MSC_VER)
#pragma warning(push)
#pragma warning(disable : 4996)
#endif // _MSC_VER
// Starting with Microsoft SDK for Windows 8.1, this function is deprecated
// in favor of the new Windows Version Helper APIs. Since we don't specify a
// minimum SDK version, it's easier to simply disable the warning rather than
// try to support both APIs.
if (GetVersionEx((LPOSVERSIONINFO)&info) == 0)
return llvm::VersionTuple();
#if defined(_MSC_VER)
#pragma warning(pop)
#endif // _MSC_VER

return llvm::VersionTuple(info.dwMajorVersion, info.dwMinorVersion, 0,
info.dwBuildNumber);
Expand Down

0 comments on commit c6fcdd3

Please sign in to comment.