So will you do a superset or should we approve this patch.<div><br></div><div>In retrospect, we should have added caching later with more full unit tests, given how complex it ends up<br><br><div class="gmail_quote"><div dir="ltr">On Thu, Apr 28, 2016, 4:51 PM George Burgess IV <<a href="mailto:george.burgess.iv@gmail.com">george.burgess.iv@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">george.burgess.iv added a comment.<br>
<br>
...And it looks like this behavior already exists in the walker anyway (because we consult the cache first). Yay.<br>
<br>
I'll take a shot at fixing it, and add tests for cases like this.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D19695" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19695</a><br>
<br>
<br>
<br>
</blockquote></div></div>