[PATCH] D18657: Propagate missing empty exception spec from function declared in system header

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 22 12:30:38 PDT 2016


rsmith added a subscriber: rsmith.

================
Comment at: lib/Sema/SemaExceptionSpec.cpp:260
@@ -261,1 +259,3 @@
+      (First->getLocation().isInvalid() ||
+       Context.getSourceManager().isInSystemHeader(First->getLocation()))) {
     New->setType(Context.getFunctionType(
----------------
rjmccall wrote:
> Please find a way to sink the call to getFirstDecl() to the point where we need it.
`getFirstDecl` is cheap these days.


http://reviews.llvm.org/D18657





More information about the cfe-commits mailing list