[lld] r289084 - Move Memory.{h, cpp} to lld/Support so that we can use them from COFF.

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 07:35:57 PST 2016


Sean Silva via llvm-commits <llvm-commits at lists.llvm.org> writes:

> Please revert this for now. The use of globals in LLD has been acceptable
> on the premise that these variables are in the "LLD is a program, not a
> library" part of LLD. Putting globals in a library is another matter and
> should not be done lightly.

Using a global allocator is fine for both the COFF and ELF lld. Having
them share the code is a good think.

Cheers,
Rafael


More information about the llvm-commits mailing list