<div dir="ltr">Sorry for the belated response<div><br></div><div>I experienced perhaps too many problems with this patch, for example, it's was too unstable (it sometimes crashes), it didn't compile in the first place, it lacked the instructions how to test it, it lacked some required file, etc. and it just took too much time even to just run it. Do you mind if I ask you to prepare a patch more carefully? Ideally it should come with a script that builds everything from source with and without profile-guided optimization to demonstrate the feature, as I partly did in my github repository for you.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 31, 2018 at 12:50 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Rafael Avila de Espindola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> writes:<br>
<br>
> Rafael Avila de Espindola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> writes:<br>
><br>
>> The patch needs to be rebased and clang formatted<br>
>><br>
>> We should start with a version that doesn't depend on the llvm side, as<br>
>> there are too many moving pieces right now.<br>
>><br>
>> I have started working on the above.<br>
><br>
> OK, I have a patch against lld trunk that I can build standalone. I will<br>
> test it with perf generated call graphs and report.<br>
<br>
</div></div>I am getting a crash when trying to link lld itself.<br>
<br>
The testcase is at<br>
<a href="https://s3-us-west-2.amazonaws.com/linker-tests/t.tar.xz" rel="noreferrer" target="_blank">https://s3-us-west-2.<wbr>amazonaws.com/linker-tests/t.<wbr>tar.xz</a>.<br>
<br>
Please take a look and upload a new version of the patch I emailed. That<br>
is: read callgrah from a text file and no dependencies on llvm changes.<br>
<br>
The error also happens with --no-threads. The valgrind reported errors<br>
are attached.<br>
<br>
<br><br>
Cheers,<br>
Rafael<br>
<br></blockquote></div><br></div>