[PATCH] D53029: [llvm-objcopy] Add -F|--target compatibility

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 15 13:09:15 PDT 2018


rupprecht added inline comments.


================
Comment at: llvm/trunk/test/tools/llvm-objcopy/input-output-target.test:14
+
+# TODO: check --target and --input-target/--output-target are incompatible
+# RUN: not llvm-objcopy --target binary --input-target binary -B i386:x86-64 \
----------------
jhenderson wrote:
> What's this TODO about?
D'oh, forgot to remove the TODO after I implemented the test case
Will remove this in a separate commit, thanks for noticing this!


Repository:
  rL LLVM

https://reviews.llvm.org/D53029





More information about the llvm-commits mailing list