The comment means space efficient, in terms of set reuse of existing sets.  Leaving it is likely to just confuse future readers <div>:)</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Aug 14, 2015, 8:45 PM Matt Arsenault <<a href="mailto:Matthew.Arsenault@amd.com" target="_blank">Matthew.Arsenault@amd.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">arsenm added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D12046#225002" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12046#225002</a>, @dberlin wrote:<br>
<br>
> Do you actually use Visited outside of the iterator (It's not visible<br>
>  in this patch if you do)?<br>
>  If not, you should just use depth_first, not depth_first_ext.<br>
<br>
<br>
No. I thought might have a use for it, but this doesn't use it yet. The comment on it also says "This iterator stores the 'visited' set in an external set, which allows it to be more efficient, and allows external clients to use the set for other purposes." so I figured if it's faster to just leave it<br>
<br>
<br>
<a href="http://reviews.llvm.org/D12046" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12046</a><br>
<br>
<br>
<br>
</blockquote></div>