Re: [PATCH] D12832: [Driver] Added support for Windows 10 SDK

2015-09-16 Thread Igor Kudrin via cfe-commits
ikudrin updated this revision to Diff 34965. ikudrin added a comment. - reworked the conditional statement - formatting http://reviews.llvm.org/D12832 Files: cfe/trunk/lib/Driver/MSVCToolChain.cpp cfe/trunk/lib/Driver/ToolChains.h Index: cfe/trunk/lib/Driver/ToolChains.h ==

Re: [PATCH] D12832: [Driver] Added support for Windows 10 SDK

2015-09-16 Thread David Majnemer via cfe-commits
majnemer added a subscriber: majnemer. Comment at: cfe/trunk/lib/Driver/MSVCToolChain.cpp:263 @@ +262,3 @@ + std::sscanf(RegistrySDKVersion.c_str(), "v%d.", &Major); + if (Major > 7 && Major < 10) { +// Windows SDK 8.x installs libraries in a folder whose names depend on the

Re: [PATCH] D12832: [Driver] Added support for Windows 10 SDK

2015-09-16 Thread Igor Kudrin via cfe-commits
ikudrin added a comment. Ping? http://reviews.llvm.org/D12832 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D12832: [Driver] Added support for Windows 10 SDK

2015-09-13 Thread Igor Kudrin via cfe-commits
ikudrin created this revision. ikudrin added reviewers: ruiu, rnk. ikudrin added a subscriber: cfe-commits. With Windows 10 SDK, Include and Lib directories now contain an additional subfolder with the name that corresponds to the full version of the SDK, for example: - C:\Program Files (x86)\