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

[Richard Smith via cfe-commits]

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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits