[PATCH] D62718: [llvm-objcopy] Change handling of output file permissions

Alex Brachet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 21 17:54:29 PDT 2019


abrachet marked 4 inline comments as done.
abrachet added inline comments.


================
Comment at: llvm/tools/llvm-objcopy/llvm-objcopy.cpp:246
+  } else {
+    Stat.permissions(static_cast<sys::fs::perms>(0775));
+  }
----------------
rupprecht wrote:
> 0777
> 
> (the umask should be what masks other/world permissions)
Great, thanks!


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

https://reviews.llvm.org/D62718





More information about the llvm-commits mailing list