[PATCH] D67656: [llvm-objcopy] Add --set-section-alignment

Jake Ehrlich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 17 11:44:16 PDT 2019


jakehehrlich added inline comments.


================
Comment at: tools/llvm-objcopy/ELF/ELFObjcopy.cpp:769
       getOutputElfType(Config.OutputArch.getValueOr(MachineInfo()));
   if (Error E = handleArgs(Config, *Obj, Reader, OutputElfType))
     return E;
----------------
I'm confused, you still go though handleArgs here. Why do you need to handle this flag above?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67656/new/

https://reviews.llvm.org/D67656





More information about the llvm-commits mailing list