[PATCH] D107249: [GlobalOpt] Don't replace alias with aliasee if either alias/aliasee may be preemptible

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Do we need to fix other places?  I tried grepping for GlobalAlias, and at least BasicAliasAnalysis.cpp and MemoryBuiltins.cpp and StackSafetyAnalysis.cpp contain similar isInterposable() checks.

In D107249#2919094 <https://reviews.llvm.org/D107249#2919094>, @efriedma wrote:

> 2. Can we forbid aliases that refer to globals that are interposable, or use weak linkage?  In theory, this prohibition doesn't cause any loss of generality in IR: we can turn a weak definition into an alias pointing to an internal definition, which should be essentially equivalent.

Any comment on this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107249