[PATCH] D114639: Raise the minimum Visual Studio version to VS2019

Michael Kruse via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 29 10:07:36 PST 2021


Meinersbur added inline comments.


================
Comment at: clang/docs/UsersManual.rst:3546
 
-    cmake -G"Visual Studio 15 2017" -T LLVM ..
+    cmake -G"Visual Studio 17 2022" -T LLVM ..
 
----------------
jhenderson wrote:
> RKSimon wrote:
> > aaron.ballman wrote:
> > > jhenderson wrote:
> > > > I think the missing space should be fixed to :)
> > > +1 to the missing space.
> > This one is confusing - it isn't in my local diff, the raw diff, or if I reapply the raw diff - it looks to have just appeared when you quote it in phab comments?
> The fact that the space is missing? It's missing in the current repo too.
It works with and without the space, like `-GNinja` and `-G Ninja` both work.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114639/new/

https://reviews.llvm.org/D114639



More information about the cfe-commits mailing list