<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 24, 2014 at 4:25 PM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@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 style="word-wrap:break-word">I don’t think it would have shown up as a leak. The bad code was adding values to a DenseMap and never clever them out between functions. I don’t see how lsan could have detected that.</div></blockquote><div><br></div><div>Yep, no way. We would need a memory profiler and compare the heap profiles between revisions to detect spikes. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div class="h5"><div><br><div><blockquote type="cite"><div>On Sep 24, 2014, at 4:23 PM, Kostya Serebryany <<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>> wrote:</div><br><div><br><br style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><div class="gmail_quote" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">On Wed, Sep 24, 2014 at 4:21 PM, Eric Christopher<span> </span><span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span><span> </span>wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div>On Wed, Sep 24, 2014 at 4:20 PM, Kostya Serebryany <<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>> wrote:<br>><br>><br>> On Wed, Sep 24, 2014 at 2:47 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>><br>> wrote:<br>>><br>>> On Wed, Sep 24, 2014 at 1:16 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>><br>>> wrote:<br>>> > On 23 September 2014 18:56, Bob Wilson <<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@apple.com</a>> wrote:<br>>> >> We’re seeing significant regressions in memory usage. I think you<br>>> >> forgot to clear the PreferredExtendType DenseMap that you added here.<br>>> ><br>>> > Seems we're missing some valgrind / sanitizer buildbots to check that<br>>> > automatically?<br>>> ><br>>><br>>><span> </span><a href="http://lab.llvm.org:8011/builders/clang-x86_64-linux-vg" target="_blank">http://lab.llvm.org:8011/builders/clang-x86_64-linux-vg</a><br>>><br>>> which appears to be having problems.<br>>><br>>> I know that Kostya has some sanitizer bots, but I'm not sure what the<br>>> status is there.<br>><br>><br>> We have two bots:<br>><span> </span><a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux</a><br>><span> </span><a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap/" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap/</a><br>> The second one is red at the moment with some (fresh?) msan reports.<br>> What is the problem here?<br>><br><br></div></div>Aha.<br><br>Anyhow as of r218101 we were leaking quite a bit of memory. I don't<br>know if the sanitizers reported it or not.<br></blockquote><div><br></div><div>lsan is silent. repro? </div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><br>-eric<br><br>>><br>>><br>>> -eric<br>><br>></blockquote></div></div></blockquote></div><br></div></div></div></div></blockquote></div><br></div></div>