[PATCH] D41097: [ELF] Only check user permissions in the file access test

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 11 15:58:12 PST 2017


ruiu added a comment.

LGTM

I believe it's still fragile against weird umask setting (such as 444), but that's in practice not an issue.

Don't you need to run this only on Unix?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D41097





More information about the llvm-commits mailing list