<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Feb 1, 2018 at 2:12 PM, Rafael Avila de Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.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"><span class="gmail-">Peter Collingbourne <<a href="mailto:peter@pcc.me.uk">peter@pcc.me.uk</a>> writes:<br>
<br>
> These are the results that I previously got with chrome:<br>
> <a href="https://bugs.chromium.org/p/chromium/issues/detail?id=759259#c10" rel="noreferrer" target="_blank">https://bugs.chromium.org/p/<wbr>chromium/issues/detail?id=<wbr>759259#c10</a><br>
<br>
</span>Cool. You mention local symbols, so I assume that you were using the<br>
older patch version that read the call graph from a text file, right?<br></blockquote><div><br></div><div>Yes.</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">
Handling ICF is a nice improvement. Would you mind sending a patch that<br>
adds that to the current version of the patch?<br></blockquote><div><br></div><div>The patch has changed significantly since I last looked at it, but at the time it was only a matter of looking through the Repl pointer. I think you should be able to replace "SymbolSection[D->getName()] = IS;" with "SymbolSection[D->getName()] = IS->Repl;" in your latest patch.</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">
Is there documentation somewhere on how to run these benchmarks? If not,<br>
would you mind running them again once ICF added to this patch?<br></blockquote><div><br></div><div>Unfortunately the documentation for running chromium benchmarks with profiling is not great. I will try to find some time soon to redo the benchmarks with the latest patch.</div><div><br></div></div>-- <br><div class="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div></div>