<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 8, 2016 at 3:19 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Thu, Sep 8, 2016 at 3:14 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">silvas added a subscriber: silvas.<br>
<span><br>
================<br>
Comment at: ELF/InputFiles.cpp:736<br>
@@ +735,3 @@<br>
+template <class ELFT> std::unique_ptr<InputFile> BinaryFile::createELF() {<br>
+  ELFCreator<ELFT> ELF(ET_REL, Config->EMachine);<br>
+  auto DataSec = ELF.addSection(".data");<br>
----------------<br>
</span><span>ruiu wrote:<br>
> Bigcheese wrote:<br>
> > ruiu wrote:<br>
> > > How about this?<br>
> > 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.<br>
> 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.<br>
</span>Like Michael already said, ELFCreator is already used in at least 2 other places for PS4. It doesn't make sense to move this BinaryFile-specific code into ELFCreator.<br></blockquote><div><br></div></span><div>Is that change for PS4 upstreamed?</div></div></div></div>
</blockquote></div><br></div><div class="gmail_extra">Not yet. Right now, we are required to keep those private, but we try to do things so that they are ready to be sent upstream if we get the approval to do so (e.g. we merge the latest LLD daily).</div><div class="gmail_extra"><br></div><div class="gmail_extra">-- Sean Silva</div></div>