[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