[llvm] [GlobalOpt] Don't replace aliasee with alias that has weak linkage (PR #91483)

via llvm-commits llvm-commits at lists.llvm.org
Thu May 16 17:07:46 PDT 2024


DianQK wrote:

> How do we get `@f2 = linkonce_odr hidden alias void (), ptr @f1` in the first place? This codegen seems problematic if `f2` can be defined in another module as something different.
> 
> `weak/linkonce` stopping the replace makes sense to me but `weak_odr/linkonce_odr` should not.

Yes. IIUC, `weak_odr/linkonce_odr` should assume that the definitions of all symbols are the same.

Do you have more information? @avikivity

https://github.com/llvm/llvm-project/pull/91483


More information about the llvm-commits mailing list