[PATCH] D20382: Add postorder support to RecursiveASTVisitor

[Benjamin Kramer via cfe-commits]

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