<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 6, 2015 at 12:39 PM, Sanjoy Das <span dir="ltr"><<a href="mailto:sanjoy@playingwithpointers.com" target="_blank">sanjoy@playingwithpointers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":28g" class="a3s" style="overflow:hidden">Another way of approaching the problem: keep the two words in<br>
DenseMapIterators even in non-asserts mode, and have the asserts mode<br>
only turn on the assertions.  I'd speculate that not many places<br>
within LLVM/Clang/LLDB keep around iterators long enough for this to<br>
impact RAM usage from the size bloat within the iterators.</div></blockquote></div><br>It has more impact than that though. It also means we need 3x the registers, 3x the function arguments, etc etc. I think this is a pretty high cost to pay.</div></div>