<div dir="ltr">I think it's worth figuring out when this is called with the semantic or syntactic version and why this can't lead to double visitation. Then add a comment while you're changing the method so the next person doesn't have to figure it all out :)</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Sep 30, 2015 at 12:15 AM Angel Garcia <<a href="mailto:angelgarcia@google.com">angelgarcia@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">angelgarcia added a comment.<br>
<br>
Yes, it breaks a few tests:<br>
<br>
FAIL: Clang :: Analysis/operator-calls.cpp (598 of 8596)<br>
<br>
FAIL: Clang :: Analysis/misc-ps-region-store.cpp (599 of 8596)<br>
<br>
FAIL: Clang :: Analysis/array-struct-region.c (602 of 8596)<br>
<br>
<br>
<a href="http://reviews.llvm.org/D13249" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13249</a><br>
<br>
<br>
<br>
</blockquote></div>