[PATCH] D30736: [Support] Add support for getting file system permissions on Windows and implement sys::fs::set/getPermissions to work with them

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 13 09:18:28 PDT 2017


jhenderson updated this revision to Diff 91571.
jhenderson retitled this revision from "[Support] Add support for getting file system permissions on Windows and implement sys::fs::permissions to set them" to "[Support] Add support for getting file system permissions on Windows and implement sys::fs::set/getPermissions to work with them".
jhenderson edited the summary of this revision.
jhenderson added a comment.

Fixed Linux unit tests by adding a new member of the perms structure to cover all valid values (except perms_not_known) and using this to mask the Perms member of the file_status struct. This removes the file type from this member, which was probably the original desired behaviour anyway. Users can get the file type from the Type member instead.


https://reviews.llvm.org/D30736

Files:
  include/llvm/Support/FileSystem.h
  lib/Support/Path.cpp
  lib/Support/Unix/Path.inc
  lib/Support/Windows/Path.inc
  unittests/Support/Path.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30736.91571.patch
Type: text/x-patch
Size: 13631 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170313/bc1a9b7d/attachment.bin>


More information about the llvm-commits mailing list