jklaehn added a comment.

In https://reviews.llvm.org/D35271#809159, @vsk wrote:

> I wonder if it's possible to do away with the calls to 'updated()'... it 
> seems strange that we initialize the same preprocessor repeatedly. Is there 
> any way to finalize an ASTInfoCollector after ReadAST happens (or 
> ASTReaderListeners in general)?


I can look into this but would prefer to do so in a different patch, as this 
would require refactoring beyond this simple bug fix. Would it be okay to land 
this patch as-is?


https://reviews.llvm.org/D35271



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

Reply via email to