<div dir="ltr">It would have been good to reference this discussion in the commit message of the revert (or even just noting that it is temporarily reverted to fix breakage), such that those of us who live downstream know whether the API change is permanent or not. </div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 14, 2016 at 10:07 AM, Michael Zolotukhin via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Rafael,<br>
<br>
I temporarily reverted r256105 to unbreak our bots in r257751. Please feel free to recommit with a fix.<br>
<br>
Thanks,<br>
Michael<br>
<div class="HOEnZb"><div class="h5"><br>
> On Jan 13, 2016, at 7:11 PM, Michael Zolotukhin <<a href="mailto:mzolotukhin@apple.com">mzolotukhin@apple.com</a>> wrote:<br>
><br>
> Hi Rafael,<br>
><br>
> We’ve started to hit this assert in our LLVM bootstraps. Do you expect this fix to be committed anytime soon, or is it better just to revert for now?<br>
><br>
> Thanks,<br>
> Michael<br>
>> On Jan 8, 2016, at 3:20 PM, Mikhail Zolotukhin <<a href="mailto:mzolotukhin@apple.com">mzolotukhin@apple.com</a>> wrote:<br>
>><br>
>> Hi Rafael,<br>
>><br>
>> I recently hit the same bug while building 483.xalanc from SPEC2006. Do you mind reverting the original patch while the fix is under review?<br>
>><br>
>> Thanks,<br>
>> Michael<br>
>><br>
>>> On Dec 22, 2015, at 7:46 AM, Rafael Espíndola via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>>><br>
>>> The asserts for checking all uses while the module is being<br>
>>> materialized found the first problem.<br>
>>><br>
>>> The issues happens when libLTO is trying to set<br>
>>> LTO_SYMBOL_SCOPE_DEFAULT_CAN_BE_HIDDEN. It needs all uses in order to<br>
>>> produce a valid answer.<br>
>>><br>
>>> The attached patch adds a graceful degradation.<br>
>>><br>
>>> What is done instead in the gold plugin is to check the uses in the<br>
>>> merged module.<br>
>>><br>
>>> If this degrades quality too much, a solution would be to check the<br>
>>> uses and add unnamed_addr marks at the end of IRGen. That way<br>
>>> canBeOmittedFromSymbolTable would not need to run analyzeGlobal at<br>
>>> all.<br>
>>><br>
>>> Cheers,<br>
>>> Rafael<br>
>>> <t.diff>_______________________________________________<br>
>>> llvm-commits mailing list<br>
>>> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
>>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
>><br>
><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>