[PATCH] D39666: ELF: Remove DefinedCommon.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 5 14:32:51 PST 2017


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM

Very nice improvement! I haven't thought of this approach before, but this totally makes sense to me.

If you need my patch, please commit it.



================
Comment at: lld/ELF/InputSection.h:57
 
+  unsigned Bss : 1;
+
----------------
Can you add this to InputSection?


https://reviews.llvm.org/D39666





More information about the llvm-commits mailing list