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

Michael Spencer via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 8 14:35:10 PDT 2016


Bigcheese 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");
----------------
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.


https://reviews.llvm.org/D24060





More information about the llvm-commits mailing list