[PATCH] D62718: [llvm-objcopy] Change output file permissions to be the same as the input file

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 6 03:35:55 PDT 2019


jhenderson added inline comments.


================
Comment at: llvm/test/tools/llvm-objcopy/ELF/invalid-preserve-dates.test:1
+# RUN: llvm-objcopy --preserve-dates %p/Inputs/alloc-symtab.o -
+# CHECK: error: --preserve-dates requires a file
----------------
This test (and the corresponding behaviour in the code) feels like it belongs in a separate patch to me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62718





More information about the llvm-commits mailing list