[PATCH] D20136: Get default -fms-compatibility-version from cl.exe's version
Adrian McCarthy via cfe-commits
cfe-commits at lists.llvm.org
Tue May 10 15:09:57 PDT 2016
amccarth added inline comments.
================
Comment at: lib/Driver/MSVCToolChain.cpp:472
@@ +471,3 @@
+
+ const DWORD VersionSize = ::GetFileVersionInfoSizeA(ClExe.c_str(), nullptr);
+ if (VersionSize == 0) {
----------------
majnemer wrote:
> Why not use the `GetFileVersionInfoSizeW` variant?
I started down that road, but it seemed overkill to convert the path to a wide string. I'm happy to do it if you think it worthwhile.
http://reviews.llvm.org/D20136
More information about the cfe-commits
mailing list