[PATCH] D85782: [X86][ELF] Prefer lowering MC_GlobalAddress operands to .Lfoo$local only for STV_DEFAULT globals

Wed Sep 16 07:28:24 PDT 2020

bd1976llvm added a comment.

In D85782#2273938 <https://reviews.llvm.org/D85782#2273938>, @hans wrote:

> In D85782#2273402 <https://reviews.llvm.org/D85782#2273402>, @bd1976llvm wrote:
>
>> @hans, I think that we should put this change onto the release branch for llvm11.
>
> I'm not familiar with the details of this patch, can you explain why we should merge it to llvm11?
>
> We're very late in the release process, so unless it's fixing some major problem, I would be hesitant to merge.

Apologies - I failed to consider where LLVM is in the release process!

This isn't needed to fix a major problem. However, it would be great to get this patch into a released LLVM ASAP to prevent users coming to rely on the behaviour prior to this patch. See report by @skan above. The patch should be very safe as it essentially restores the referencing behaviour to what it was prior to https://reviews.llvm.org/D73230 for symbol definitions that do not have default ELF visibility.

If you have to do another RC it would be worth considering merging this.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85782/new/

https://reviews.llvm.org/D85782