Re: [PATCH] D19231: [ASTMatchers] Do not try to memoize nodes we can't compare.

2016-04-19 Thread Samuel Benzaquen via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL266748: [ASTMatchers] Do not try to memoize nodes we can't compare. (authored by sbenza). Changed prior to commit: http://reviews.llvm.org/D19231?vs=54089&id=54207#toc Repository: rL LLVM http://rev

Re: [PATCH] D19231: [ASTMatchers] Do not try to memoize nodes we can't compare.

2016-04-18 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. Looks good. Thanks! http://reviews.llvm.org/D19231 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/

[PATCH] D19231: [ASTMatchers] Do not try to memoize nodes we can't compare.

2016-04-18 Thread Samuel Benzaquen via cfe-commits
sbenza created this revision. sbenza added a reviewer: alexfh. sbenza added a subscriber: cfe-commits. Herald added a subscriber: klimek. Prevent hasAncestor from comparing nodes that are not supported. hasDescendant was fixed some time ago to avoid this problem. I'm applying the same fix to hasAn