vsapsai added inline comments.

================
Comment at: clang/test/Modules/dependency-file-symlinks.c:3
+
+// RUN: rm -fr %t
+// RUN: mkdir -p %t/cache
----------------
Oh, and can you please change the order of flags, so it is `rm -rf %t`? The 
real reason is my personal preference but `-rf` is actually more consistent 
with existing tests, it is used more than 10x more often than `-fr`.


Repository:
  rC Clang

https://reviews.llvm.org/D53522



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to