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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits