[PATCH] D53522: [Frontend] Include module map header declaration in dependency file output
Volodymyr Sapsai via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 23 14:48:06 PDT 2018
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
More information about the cfe-commits
mailing list