<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 18, 2014 at 7:20 AM, Benjamin Kramer <span dir="ltr"><<a href="mailto:benny.kra@gmail.com" target="_blank">benny.kra@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb adM"><div class="">> +  /// \brief Counter for the next DFS number to assign.<br>
> +  int NextDFSNumber;<br>
<br>
</div></div>This member var isn't initialized anywhere. MSan and valgrind are upset.<br></blockquote><div><br></div><div>Ow, good catch.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><div class="h5">> LazyCallGraph::LazyCallGraph(LazyCallGraph &&G)<br>
>     : BPA(std::move(G.BPA)), EntryNodes(std::move(G.EntryNodes)),<br>
> -      EntryNodeSet(std::move(G.EntryNodeSet)) {<br>
> +      EntryNodeSet(std::move(G.EntryNodeSet)), SCCBPA(std::move(G.SCCBPA)),<br>
> +      SCCMap(std::move(G.SCCMap)), LeafSCCs(std::move(G.LeafSCCs)),<br>
> +      DFSStack(std::move(G.DFSStack)),<br>
> +      SCCEntryNodes(std::move(G.SCCEntryNodes)) {<br>
<br>
</div></div>The ctor misses two members, NodeMap and NextDFSNumber. Intentional?<br>
<br>
Same for the move assignment operator.<br></blockquote></div><br></div><div class="gmail_extra">Doh! Definitely a bug.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Committed fixes to both in r206646. Thanks for the review!</div>
</div>