[patch] Fix pr25919

[Mikhail Zolotukhin via llvm-commits]

Hi Rafael,

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?

Thanks,
Michael

> On Dec 22, 2015, at 7:46 AM, Rafael Espíndola via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> The asserts for checking all uses while the module is being
> materialized found the first problem.
> 
> The issues happens when libLTO is trying to set
> LTO_SYMBOL_SCOPE_DEFAULT_CAN_BE_HIDDEN. It needs all uses in order to
> produce a valid answer.
> 
> The attached patch adds a graceful degradation.
> 
> What is done instead in the gold plugin is to check the uses in the
> merged module.
> 
> If this degrades quality too much, a solution would be to check the
> uses and add unnamed_addr marks at the end of IRGen. That way
> canBeOmittedFromSymbolTable would not need to run analyzeGlobal at
> all.
> 
> Cheers,
> Rafael
> <t.diff>_______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits