[PATCH] D14387: Move comdat setup after global value body linking in ModuleLinker

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 6 11:20:09 PST 2015


> Can you change metadata linking *not* to cause the def to be linked in?
> The debug info *shouldn't* be causing anything to be linked in that isn't
> already (since debug info should not affect code generation).  Instead,
> you can just have the `variable:` field point at nullptr.

+1

Adding metadata should not change what non-metadata IR is linked in
(even declarations).

Cheers,
Rafael


More information about the llvm-commits mailing list