[PATCH] D52843: Update Clang Windows getting started docs
Zachary Turner via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 3 16:02:17 PDT 2018
zturner added inline comments.
================
Comment at: clang/www/get_started.html:246
+ <tt>"C:\Program Files (x86)\Microsoft Visual
+ Studio\2017\Professional\VC\Auxiliary\Build\vcvarsall.bat" x64</tt>
+ </li>
----------------
zturner wrote:
> STL_MSFT wrote:
> > This assumes the Professional SKU; perhaps the instructions should be for Community?
> Would something like `%VS2017INSTALLDIR%` be even better?
Granted it assumes 2017, but not as bad as assuming both Professional SKU + default installation path.
https://reviews.llvm.org/D52843
More information about the cfe-commits
mailing list