[PATCH] D70854: [Clang] In tests, do not always assume others permissions are set

Alexandre Ganea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 11 12:46:24 PST 2019


aganea added a comment.

Yes this was added in rG18627115f4d2db5dc73207e0b5312f52536be7dd <https://reviews.llvm.org/rG18627115f4d2db5dc73207e0b5312f52536be7dd> and rGe08b59f81d950bd5c8b8528fcb3ac4230c7b736c <https://reviews.llvm.org/rGe08b59f81d950bd5c8b8528fcb3ac4230c7b736c>. It looks like it was only there to validate that specific refactoring.
Should I remove these two files, or simply commit the previous diff? (ie. -rw-rw----)


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

https://reviews.llvm.org/D70854





More information about the llvm-commits mailing list