<div dir="ltr"><div class="gmail_extra"><div><div class="gmail_signature" data-smartmail="gmail_signature">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></div></div><div class="gmail_quote"><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><div class="gmail_extra">I wasn't able to duplicate the crash at all (Windows or Linux, debug with asserts), but from the log it I'm pretty sure it was related the issue I working on. Here's an updated patch.</div><div class="gmail_extra"><br></div><div class="gmail_extra">

<span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">- Michael Spencer</span>

<br></div></div>