<div dir="ltr">How was the bisecting done -- rebuild lld each time with IR file unchanged? The IR file generated could already be bad.<div><br></div><div>David</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 13, 2019 at 12:21 AM Hans Wennborg via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Did you look at<br>
<a href="https://bugs.chromium.org/p/chromium/issues/detail?id=992871#c3" rel="noreferrer" target="_blank">https://bugs.chromium.org/p/chromium/issues/detail?id=992871#c3</a> ? It<br>
shows how to reproduce using the tarball on Linux.<br>
<br>
On Mon, Aug 12, 2019 at 8:17 PM Carrot Wei <<a href="mailto:carrot@google.com" target="_blank">carrot@google.com</a>> wrote:<br>
><br>
> The MBP patch doesn't touch debug information, I suspect there is<br>
> something unrelated was triggered.<br>
> I tried your tarball, but found it is for windows, I don't have a<br>
> windows machine :(. Do you have a linux reproduce? It's even better to<br>
> have a compiler command line.<br>
><br>
> On Mon, Aug 12, 2019 at 7:24 AM Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
> ><br>
> > I've reverted this in r368579 since it causes asserts when building<br>
> > Chromium. See <a href="https://bugs.chromium.org/p/chromium/issues/detail?id=992871" rel="noreferrer" target="_blank">https://bugs.chromium.org/p/chromium/issues/detail?id=992871</a><br>
> > for details and reproducer.<br>
> ><br>
> > On Thu, Aug 8, 2019 at 10:24 PM Guozhi Wei via llvm-commits<br>
> > <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> > ><br>
> > > Author: carrot<br>
> > > Date: Thu Aug  8 13:25:23 2019<br>
> > > New Revision: 368339<br>
> > ><br>
> > > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=368339&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=368339&view=rev</a><br>
> > > Log:<br>
> > > [MBP] Disable aggressive loop rotate in plain mode<br>
> > ><br>
> > > Patch <a href="https://reviews.llvm.org/D43256" rel="noreferrer" target="_blank">https://reviews.llvm.org/D43256</a> introduced more aggressive loop layout optimization which depends on profile information. If profile information is not available, the statically estimated profile information(generated by BranchProbabilityInfo.cpp) is used. If user program doesn't behave as BranchProbabilityInfo.cpp expected, the layout may be worse.<br>
> > ><br>
> > > To be conservative this patch restores the original layout algorithm in plain mode. But user can still try the aggressive layout optimization with -force-precise-rotation-cost=true.<br>
> > ><br>
> > > Differential Revision: <a href="https://reviews.llvm.org/D65673" rel="noreferrer" target="_blank">https://reviews.llvm.org/D65673</a><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>