<div dir="ltr"><div class="gmail_extra">Thanks for tracking this down. LGTM with a style nit below.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 29, 2013 at 12:59 PM, Howard Hinnant <span dir="ltr"><<a href="mailto:hhinnant@apple.com" target="_blank" class="cremed">hhinnant@apple.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=":bst" style="overflow:hidden">Index: include/llvm/ADT/DenseMap.h<br>
===================================================================<br>
--- include/llvm/ADT/DenseMap.h (revision 193642)<br>
+++ include/llvm/ADT/DenseMap.h (working copy)<br>
@@ -439,8 +439,8 @@<br>
       LookupBucketFor(Key, TheBucket);<br>
       NumBuckets = getNumBuckets();<br>
     }<br>
-    if (NumBuckets-(NewNumEntries+getNumTombstones()) <= NumBuckets/8) {<br>
-      this->grow(NumBuckets * 2);<br>
+    else if (NumBuckets-(NewNumEntries+getNumTombstones()) <= NumBuckets/8) {<br></div></blockquote><div><br></div><div>Pull the else onto the same line as the }?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":bst" style="overflow:hidden">
+      this->grow(NumBuckets);<br>
       LookupBucketFor(Key, TheBucket);<br>
     }<br>
     assert(TheBucket);<br>
<br>
Howard</div></blockquote></div><br><br></div></div>