bkramer added a comment.

Having postorder traversal makes sense to me. The thing I'm worried about is 
how much this will bloat object code. RecursiveASTVisitor is already a major 
contributor to the size of clang's binary and we've hit issues with it in the 
past (hitting .obj size limits on Windows for example) can you show numbers of 
the size of clang before and after this change? Ideally in Release and Debug 
configurations.

If it grows it too much we'll have to find another way of enabling/disabling it.


http://reviews.llvm.org/D20382



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

Reply via email to