r.stahl added a comment.

In https://reviews.llvm.org/D47698#1140629, @thakis wrote:

> This code is live when reading pchs, correct? Does this have any measurable 
> perf impact on deserializing pchs for, say, Cocoa.h or Windows.h?


I don't know for sure, but it should be used - yes. I have not measured a 
possible performance impact. Do you have a suggestion how I could do this on a 
Linux setup?

Note that I did not implement this as nice-to-have feature, but for fixing a 
concrete issue: https://reviews.llvm.org/D26054#1085413



================
Comment at: unittests/AST/ASTImporterTest.cpp:1534
+}
+TEST_P(ASTImporterTestBase, ImportSourceLocs) {
+  Decl *FromTU = getTuDecl(
----------------
balazske wrote:
> This test causes every case for expansion (macro, macro arg) to be executed 
> at import?
Yes, the last DeclRef will be "arg" on the RHS which is a macro argument 
expansion and the last IntegerLiteral will be the "1" which is a non-argument 
macro expansion.


Repository:
  rC Clang

https://reviews.llvm.org/D47698



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

Reply via email to