rjmccall added a comment.

IIRC, isInSystemHeader isn't a terribly cheap check.  Please extract this into 
a function that only needs to be called if the other checks succeed.

You should also be able to just ask about the first declaration.  There are 
contrived situations where isInSystemHeader can be false for the first 
declaration and true for later ones, but we don't need to fall over ourselves 
supporting them.


http://reviews.llvm.org/D18657



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

Reply via email to