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

Manuel Klimek klimek at google.com
Wed Feb 27 00:05:39 PST 2013


Doug, I'll not submit before I get an ok from you :)


On Tue, Feb 26, 2013 at 6:05 PM, Jordan Rose <jordan_rose at apple.com> wrote:

>
>   Sorry, I haven't commented because it looks like it's going to be less
> and less relevant to the analyzer. As much as I don't like having two
> similar data structures floating around, Edwin's right that we don't need
> to try to optimize for two different use cases if it means that we won't
> get optimal behavior in either.
>
>   This design is at least not incompatible with what the analyzer needs,
> in case we can ever unify them in the future. We may want to rename the
> analyzer's class StmtParentMap, and migrate the other users
> (migrators/translators) to AST's ParentMap.
>
> http://llvm-reviews.chandlerc.com/D267
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130227/4c613a44/attachment.html>


More information about the cfe-commits mailing list