[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