[PATCH] D38123: [driver] [cl] Add/fix c++17/c++latest

[Martell Malone via Phabricator via cfe-commits]

martell accepted this revision.
martell added a comment.
This revision is now accepted and ready to land.

It seems that msvc enabled some c++17 features when in c++14 mode and they left them enabled because projects became dependant on them.
switching to c++17 as the default and removing the c++17 features from c++14 mode seems like the correct thing to do here even if MS still use c++14 as their default.
That should be part of a different patch though because a discussion needs to be had on that issue.

LGTM as is.
Thanks for the contribution.


https://reviews.llvm.org/D38123