<div dir="ltr">On Wed, Jun 19, 2013 at 5:38 PM, Edwin Vane <span dir="ltr"><<a href="mailto:edwin.vane@intel.com" target="_blank">edwin.vane@intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
================<br>
Comment at: lib/ASTMatchers/ASTMatchFinder.cpp:388<br>
@@ +387,3 @@<br>
+    if (ResultCache.size() > MaxMemoizationEntries)<br>
+      ResultCache.clear();<br>
+    std::pair<MemoizationMap::iterator, bool> InsertResult =<br>
----------------<br>
Is it worth adding a FIXME to look into other cache ejection strategies? Would it make a difference for performance?<br></blockquote><div><br></div><div style>I already have a fixme with the definition of MaxMemoizationEntries... I don't think it'll make much of a difference though. But we'll need some more micro-benchmarks ...</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D992" target="_blank">http://llvm-reviews.chandlerc.com/D992</a><br>
<br>
BRANCH<br>
  explode-review<br>
<br>
ARCANIST PROJECT<br>
  clang<br>
</div></div></blockquote></div><br></div></div>