================
@@ -6627,6 +6627,7 @@ void Sema::checkClassLevelDLLAttribute(CXXRecordDecl
*Class) {
auto *Ctor = dyn_cast<CXXConstructorDecl>(MD);
if ((MD->isMoveAssignmentOperator() ||
(Ctor && Ctor->isMoveConstructor())) &&
+ getLangOpts().isCompatibleWithMSVC(LangOptions::MSVC2010) &&
----------------
bd1976bris wrote:
This tripped me up when I first looked at it. I think it would be nice to
introduce a no argument form `bool isCompatibleWithMSVC() const;` and then
write the MSVC compatibility bit as: `getLangOpts().isCompatibleWithMSVC() &&
!getLangOpts().isCompatibleWithMSVC(LangOptions::MSVC2015)`
https://github.com/llvm/llvm-project/pull/168170
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
