[PATCH] Add option to switch off putting header modules into the dependency file.

Daniel Jasper djasper at google.com
Thu Mar 19 02:58:08 PDT 2015


Looks good.


================
Comment at: test/Driver/pch-deps.c:18
@@ +17,3 @@
+
+// RUN: %clang -x c++ %s -o %t.o -MMD -MT dependencies -MF %t.d -fmodule-file-deps -### 2> %t
+// RUN: FileCheck %s -check-prefix=CHECK-EXPLICIT-NOPCH -input-file=%t
----------------
nit: I'd either line wrap both of these command lines (slightly preferable) or none of them.

http://reviews.llvm.org/D8378

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list