[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