[PATCH] D24060: [lld][ELF] Add support for -b binary

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 8 14:39:51 PDT 2016


ruiu added inline comments.

================
Comment at: ELF/InputFiles.cpp:736
@@ +735,3 @@
+template <class ELFT> std::unique_ptr<InputFile> BinaryFile::createELF() {
+  ELFCreator<ELFT> ELF(ET_REL, Config->EMachine);
+  auto DataSec = ELF.addSection(".data");
----------------
Bigcheese wrote:
> ruiu wrote:
> > How about this?
> Well, the code doesn't belong in ELFCreator.cpp. I can make a BinaryFile.cpp for it, but it's only 40 lines of code.
That distinction doesn't make much sense to me because this is part of the linker. If you have a concrete plan to move ELFCreator to some library, it may make sense, but it doesn't seem to happen soon.


https://reviews.llvm.org/D24060





More information about the llvm-commits mailing list