<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 20, 2018 at 2:37 PM, Easwaran Raman <span dir="ltr"><<a href="mailto:eraman@google.com" target="_blank">eraman@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I don't see D42311 as going in a different direction than this one. My patch extends GraphTraits to iterate over the outgoing edges of  node while this patch specializes CallGraphTraits for ModuleSummaryIndex.</div></blockquote><div><br></div><div>Ok, I misunderstood based on the description there and your comments about duplicating some of the summary info in D42212. Presumably then you can use this patch to do the necessary SCC based propagation of relative frequencies during the thin link then.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-m_-2482146176820097731HOEnZb"><div class="gmail-m_-2482146176820097731h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 19, 2018 at 8:43 PM, Teresa Johnson via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">tejohnson added a comment.<br>
<br>
Sorry, I missed these being updated. Will take another pass through but it looks like you have addressed my comments.<br>
<br>
Easwaran - can this be used to do the call graph walks you need for synthetic counts propagation? If not we need to figure out why, because I see you are going in a different direction in <a href="https://reviews.llvm.org/D42311" rel="noreferrer" target="_blank">https://reviews.llvm.org/D4231<wbr>1</a>.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D36311" rel="noreferrer" target="_blank">https://reviews.llvm.org/D3631<wbr>1</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail-m_-2482146176820097731gmail_signature"><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top:2px solid rgb(213,15,37)">Teresa Johnson |</td><td nowrap style="border-top:2px solid rgb(51,105,232)"> Software Engineer |</td><td nowrap style="border-top:2px solid rgb(0,153,57)"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top:2px solid rgb(238,178,17)"> <a href="tel:(408)%20460-2413" value="+14084602413" target="_blank">408-460-2413</a></td></tr></tbody></table></span></div>
</div></div>