<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Sep 8, 2016 at 9:19 AM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">ruiu added inline comments.<br>
<br>
================<br>
Comment at: ELF/CMakeLists.txt:20<br>
@@ -19,2 +19,3 @@<br>
   ScriptParser.cpp<br>
+  SimpleELFWriter.cpp<br>
   Strings.cpp<br>
----------------<br>
I still don't like this file name. `SimpleELFWriter` sounds like it is for -o binary instead of -b. Can you rename BinaryFile.cpp?<br></blockquote><div><br></div><div>I'm fine with finding another name, but BinaryFile isn't at all what this class is.</div><div><br></div><div>- Michael Spencer<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
================<br>
Comment at: ELF/InputFiles.cpp:736<br>
@@ +735,3 @@<br>
+template <class ELFT> std::unique_ptr<InputFile> BinaryFile::createELF() {<br>
<span class="gmail-">+  SimpleELFWriter<ELFT> ELF(ET_REL, Config->EMachine);<br>
+  auto DataSec = ELF.addSection(".data");<br>
----------------<br>
</span>Please move these details to SimpleELFWriter.cpp.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D24060" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24060</a><br><br></blockquote></div></div></div>