[PATCH] D20136: Get default -fms-compatibility-version from cl.exe's version

David Majnemer via cfe-commits cfe-commits at lists.llvm.org
Fri May 13 10:32:15 PDT 2016


majnemer added inline comments.

================
Comment at: lib/Driver/MSVCToolChain.cpp:481
@@ +480,3 @@
+
+  std::vector<uint8_t> VersionBlock(VersionSize);
+  if (!::GetFileVersionInfoW(ClExeWide.c_str(), 0, VersionSize,
----------------
It might be nicer to use a `SmallVector<uint8_t, sizeof(VS_FIXEDFILEINFO)>`, or whatever `VersionSize` typically is, here to avoid heap allocation in the common case.


http://reviews.llvm.org/D20136





More information about the cfe-commits mailing list