[PATCH] Completely revamp node binding for AST matchers.

Edwin Vane edwin.vane at intel.com
Wed Jun 19 08:38:19 PDT 2013



================
Comment at: lib/ASTMatchers/ASTMatchFinder.cpp:388
@@ +387,3 @@
+    if (ResultCache.size() > MaxMemoizationEntries)
+      ResultCache.clear();
+    std::pair<MemoizationMap::iterator, bool> InsertResult =
----------------
Is it worth adding a FIXME to look into other cache ejection strategies? Would it make a difference for performance?


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

BRANCH
  explode-review

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list