[PATCH] D67590: Properly ignore mismatched exception specifiers in MSVC Compat mode.

2019-09-16 Thread Alex Fusco via Phabricator via cfe-commits
alexfusco added inline comments. Comment at: clang/test/SemaCXX/ms-exception-spec.cpp:2 +// RUN: %clang_cc1 -std=c++11 %s -fsyntax-only -verify -fms-compatibility -fexceptions -fcxx-exceptions +// RUN: %clang_cc1 -std=c++17 %s -fsyntax-only -verify -fms-compatibility

[PATCH] D67590: Properly ignore mismatched exception specifiers in MSVC Compat mode.

2019-09-16 Thread Alex Fusco via Phabricator via cfe-commits
alexfusco updated this revision to Diff 220367. alexfusco added a comment. Updated to unconditionally ignore the exception spec here, since it has already been compared. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67590/new/ https://reviews.llvm.org/D67590

[PATCH] D67590: Properly ignore mismatched exception specifiers in MSVC Compat mode.

2019-09-14 Thread Alex Fusco via Phabricator via cfe-commits
alexfusco created this revision. alexfusco added a project: clang. Herald added a subscriber: cfe-commits. This fixes the error compiling _com_ptr_t described in https://bugs.llvm.org/show_bug.cgi?id=42842 In MSVCCompat mode, clang already attempts to downgrade these mismatches to a