[PATCH] D100322: [ConstantMerge] Don't merge thread_local constants with non-thread_local constants

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

In D100322#2723566 <https://reviews.llvm.org/D100322#2723566>, @Amanieu wrote:

> I'm not even sure what it would *mean* to have an unnamed TLS constant: I would expect an earlier pass to convert it to a normal constant rather than having to it in ConstantMerge.
>
> Also, AFAIK neither clang nor rustc will ever generate unnamed TLS constants.

Makes sense, thanks!
Seems like Eli's suggestion would be best then?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100322