majnemer added a subscriber: majnemer.
majnemer requested changes to this revision.
majnemer added a reviewer: majnemer.
majnemer added a comment.
This revision now requires changes to proceed.
This requires an associated test case.
================
Comment at: lib/Sema/SemaDecl.cpp:14354-14355
@@ -14353,2 +14353,4 @@
if (!isRepresentableIntegerValue(Context, EnumVal, EltTy)) {
- if (getLangOpts().MSVCCompat) {
+ if (getLangOpts().MSVCCompat ||
+ Context.getTargetInfo().getCXXABI().isMicrosoft()) {
+ // Treat as an extension in MSVC compat mode or if using the
----------------
I would remove the `getLangOpts().MSVCCompat`.
http://reviews.llvm.org/D22273
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
