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

John McCall via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 22 13:18:18 PDT 2016


rjmccall added inline comments.

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


http://reviews.llvm.org/D18657





More information about the cfe-commits mailing list