[PATCH] D15553: PR25717: fatal IO error writing large outputs to console on Windows
Aaron Ballman via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 17 12:44:48 PST 2015
aaron.ballman added inline comments.
================
Comment at: lib/Support/raw_ostream.cpp:62
@@ +61,3 @@
+#define NOMINMAX
+#include <windows.h>
+#endif
----------------
Please include WindowsSupport.h instead of doing this.
================
Comment at: lib/Support/raw_ostream.cpp:572
@@ +571,3 @@
+static bool IsWindows7OrEarlier() {
+ unsigned Version = ::GetVersion();
+ unsigned MajorVersion = (Version & 0xFF);
----------------
GetVersion() is deprecated, you should be using IsWindows8OrGreater() instead (https://msdn.microsoft.com/en-us/library/windows/desktop/dn424961(v=vs.85).aspx).
http://reviews.llvm.org/D15553
More information about the llvm-commits
mailing list