[PATCH] D33388: [libcxx] fixup bootstrapping for mingw-w64

[NAKAMURA Takumi via Phabricator via llvm-commits]

chapuni added inline comments.


================
Comment at: libcxx/trunk/cmake/config-ix.cmake:53
+    list(APPEND CMAKE_REQUIRED_LIBRARIES ${MINGW_LIBRARIES})
+    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -D_WIN32_WINNT=0x0600")
   endif()
----------------
EricWF wrote:
> `-D_WIN32_WINNT=0x0600`. What? Why?
> 
> First isn't `0x0600` Windows Vista? Why are we asking to support such an old operating system?
> Why are we defining this at all?
> 
I think it'd be good thing to require minimum version as the bottom line.


Repository:
  rL LLVM

https://reviews.llvm.org/D33388