[PATCH] D62817: [test][llvm-objcopy] Test llvm-objcopy with standard streams

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 4 04:01:39 PDT 2019


jhenderson accepted this revision.
jhenderson added a comment.

LGTM, with one nit.



================
Comment at: llvm/test/tools/llvm-objcopy/ELF/standard-streams.test:7
+# RUN: cmp %t1 %t2
+
----------------
Delete this blank line. There should be a single trailing new line character at the end of the last line in the file with content, but nothing after that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62817





More information about the llvm-commits mailing list