[PATCH] D55039: [sema] Warn of mangling change if function parameters are noexcept.

Matt Davis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 3 16:51:38 PST 2018


mattd updated this revision to Diff 176515.
mattd added a comment.

- Move the HasNoexcept lambda to its own static function.
- Added an additional check in HasNoexcept to recursively check return values.
- Modified the parameter check in Sema::CheckFunctionDeclaration to use llvm::any_of, all we need to know is if any of the parameters might have a mangling change.
- Updated the test to include the example provided by @Rakete1111.


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

https://reviews.llvm.org/D55039

Files:
  lib/Sema/SemaDecl.cpp
  test/CXX/except/except.spec/p2-places.cpp
  test/SemaCXX/cxx1z-noexcept-function-type.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55039.176515.patch
Type: text/x-patch
Size: 6168 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181204/9805cd60/attachment.bin>


More information about the cfe-commits mailing list