constants should be tagged with unnamed_addr to merge them

[thestinger]

LLVM only exposes setUnnamedAddr through the C++ API, sadly.

Marking constants this way allows LLVM to merge the representations, because it informs the compiler that the uniqueness of the address is unimportant.

If you absolutely must have a unique address, it should probably be expressed with an attribute.

[bstrie]

Nominating for Maturity 5, Production Ready.

[thestinger]

We probably can't really do this since the addresses are significant. I'll open more specific issues.

[sanxiyn]

#6787 exposed unnamed_addr, which should help this issue.

[alexcrichton]

I'm adding an attribute for doing this in #8421