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

Samuel Benzaquen via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 18 11:33:41 PDT 2016


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 hasAncestor: if any object in the Builder map is
not comparable, skip the cache.

http://reviews.llvm.org/D19231

Files:
  lib/ASTMatchers/ASTMatchFinder.cpp
  unittests/ASTMatchers/ASTMatchersTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19231.54089.patch
Type: text/x-patch
Size: 4576 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160418/12fb58cd/attachment.bin>


More information about the cfe-commits mailing list