[PATCH] D134468: [Driver] Ignore -fmsc-version= -fms-compatibility-version= values smaller than 19

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 26 07:33:00 PDT 2022


hans added a comment.

In D134468#3812846 <https://reviews.llvm.org/D134468#3812846>, @MaskRay wrote:

> I am not a Windows driver user. This patch is entirely motivated by Windows discussions in D131465 <https://reviews.llvm.org/D131465> and I want to make `/std:` default rule simple (say, clang 16 uses C++17 for every mode for Windows). If the `-fno-threadsafe-statics` compatibility change is considered too risky, perhaps I'll just change this patch to just do C++17.

If the D131465 <https://reviews.llvm.org/D131465> discussion decides to bump to c++17, I think we should just do that bump and not drop the other compatibility stuff.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134468



More information about the cfe-commits mailing list