balazske added a comment.

I discovered that there are problems with this change. The number of loaded AST 
units is not incremented correctly and the "CTU loaded AST file" is displayed 
for every access to a AST unit (even if it was got from cache). The problem is 
that the counting and print of AST loads does not handle the case when the load 
returned a cached result (and no real loading happened). This causes "annoying" 
messages and the AST unit limit is not handled correctly.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64753/new/

https://reviews.llvm.org/D64753



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

Reply via email to