I haven't had an opportunity to look at this. I will look at it today. Please don't nlland it just yet.<br><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 23, 2018, 1:45 AM James Henderson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jhenderson accepted this revision.<br>
jhenderson added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
Much better, thanks!<br>
<br>
LGTM, although I do have one query about raw binary input. Should this go through a similar dispatch mechanism to enable other file formats to read in raw binary?<br>
<br>
<br>
<br>
================<br>
Comment at: llvm-objcopy.cpp:655<br>
     FileBuffer FB(Config.OutputFilename);<br>
-    BinaryReader Reader(Config.BinaryArch, BufOrErr->get());<br>
-    executeElfObjcopyOnBinary(Config, Reader, FB,<br>
-                              getOutputElfType(Config.BinaryArch));<br>
+    elf::executeObjcopyOnRawBinary(Config, *BufOrErr->get(), FB);<br>
   } else {<br>
----------------<br>
It seems a bit odd still having mention of elf at this level. Is it allowed to read binary input for COFF or Mach-O output?<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D53311" rel="noreferrer" target="_blank">https://reviews.llvm.org/D53311</a><br>
<br>
<br>
<br>
</blockquote></div>