[PATCH] D12695: [Driver] Use UniversalCRT on Windows if available
Reid Kleckner via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 10 11:17:01 PDT 2015
rnk added a comment.
There's a bunch of whitespace issues that clang-format can resolve. We also have the convention that variables are StudlyCaps, which isn't followed in a few places.
Do you need someone to commit this for you? If so, I can patch this in, test it manually, and deal with the style stuff if you deal with the VSDir part.
================
Comment at: lib/Driver/MSVCToolChain.cpp:293
@@ +292,3 @@
+// specific header files. If not, they are probably shipped with Universal CRT.
+bool clang::driver::toolchains::MSVCToolChain::useUniversalCRT() const
+{
----------------
This should really take VSDir as a parameter instead of recomputing it, since all the callers are already computing it.
http://reviews.llvm.org/D12695
More information about the cfe-commits
mailing list