rnk added a comment.

I don't see the MSVC build requirement as being related to the version of MSVC 
that clang-cl supports targeting. Those can be separate. That said, this might 
be the right time to sunset this compatibility logic.

If we do this, I'd like it to be a policy change. We should add a release note 
about dropping support for `-fmsc-version=N` for N<1900, and audit for other 
logic like this:
https://github.com/llvm/llvm-project/blob/main/clang/lib/Sema/SemaDeclCXX.cpp#L6390

Bonus points for a warning when the user passes 
`-fms-compatibility-version=18.*`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134468

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to