[PATCH] First step towards adding a parent map to the ASTContext.

Manuel Klimek klimek at google.com
Tue Feb 5 05:04:24 PST 2013



================
Comment at: include/clang/AST/ASTContext.h:419
@@ +418,3 @@
+  /// expensive to create (for example, when using precompiled header
+  /// preambles. Thus, there are good opportunities for optimization here.
+  /// One idea is to walk the given node downwards, looking for references
----------------
Joey Gouly wrote:
> include/clang/AST/ASTContext.h:419:14 error: missing ')'
> 
> include/clang/AST/ASTContext.h:419:14 note:
> /// preambles. Thus, there are good opportunities for optimization here.
>                    ')'
>                    ^
:)

Done.


http://llvm-reviews.chandlerc.com/D267



More information about the cfe-commits mailing list