[PATCH] D66770: Move EH spec mismatches under -fms-compatibility

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 26 15:48:16 PDT 2019


rnk created this revision.
rnk added a reviewer: hans.
Herald added a project: clang.

-fms-extensions is intended to enable conforming language extensions and
-fms-compatibility is intended to language rule relaxations, so a user
could plausibly compile with -fno-ms-compatibility on Windows while
still using dllexport, for example.  This exception specification
validation behavior has been handled as a warning since before
-fms-compatibility was added in 2011. I think it's just an oversight
that it hasn't been moved yet.

This will help users find conformance issues in their code such as those
found in _com_ptr_t as described in https://llvm.org/PR42842.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D66770

Files:
  clang/lib/Sema/SemaExceptionSpec.cpp
  clang/test/CodeGenCXX/pr18661.cpp
  clang/test/SemaCXX/MicrosoftCompatibility.cpp
  clang/test/SemaCXX/MicrosoftExtensions.cpp
  clang/test/SemaCXX/ms-exception-spec.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66770.217255.patch
Type: text/x-patch
Size: 6069 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190826/04dcdf4e/attachment.bin>


More information about the cfe-commits mailing list