<p dir="ltr">As a driver flag, this should have a -f form as well as the -fno- form.</p>
<div class="gmail_quote">On 17 Mar 2015 6:42 am, "Manuel Klimek" <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi rsmith, djasper,<br>
<br>
<a href="http://reviews.llvm.org/D8378" target="_blank">http://reviews.llvm.org/D8378</a><br>
<br>
Files:<br>
  include/clang/Driver/Options.td<br>
  lib/Driver/Tools.cpp<br>
  test/Driver/pch-deps.c<br>
<br>
Index: include/clang/Driver/Options.td<br>
===================================================================<br>
--- include/clang/Driver/Options.td<br>
+++ include/clang/Driver/Options.td<br>
@@ -770,6 +770,8 @@<br>
   Flags<[DriverOption]>;<br>
 def fimplicit_modules : Flag <["-"], "fimplicit-modules">, Group<f_Group>,<br>
   Flags<[DriverOption]>;<br>
+def fno_module_file_deps : Flag <["-"], "fno-module-file-deps">, Group<f_Group>,<br>
+  Flags<[DriverOption]>;<br>
 def fno_ms_extensions : Flag<["-"], "fno-ms-extensions">, Group<f_Group>;<br>
 def fno_ms_compatibility : Flag<["-"], "fno-ms-compatibility">, Group<f_Group>;<br>
 def fno_delayed_template_parsing : Flag<["-"], "fno-delayed-template-parsing">, Group<f_Group>;<br>
Index: lib/Driver/Tools.cpp<br>
===================================================================<br>
--- lib/Driver/Tools.cpp<br>
+++ lib/Driver/Tools.cpp<br>
@@ -325,7 +325,8 @@<br>
         A->getOption().matches(options::OPT_MD))<br>
       CmdArgs.push_back("-sys-header-deps");<br>
<br>
-    if (isa<PrecompileJobAction>(JA))<br>
+    if (isa<PrecompileJobAction>(JA) &&<br>
+        !Args.hasArg(options::OPT_fno_module_file_deps))<br>
       CmdArgs.push_back("-module-file-deps");<br>
   }<br>
<br>
Index: test/Driver/pch-deps.c<br>
===================================================================<br>
--- test/Driver/pch-deps.c<br>
+++ test/Driver/pch-deps.c<br>
@@ -8,3 +8,8 @@<br>
 // RUN: FileCheck %s -check-prefix=CHECK-NOPCH -input-file=%t<br>
 // CHECK-NOPCH: -dependency-file<br>
 // CHECK-NOPCH-NOT: -module-file-deps<br>
+<br>
+// RUN: %clang -x c-header %s -o %t.pch -MMD -MT dependencies -MF %t.d -fno-module-file-deps -### 2> %t<br>
+// RUN: FileCheck %s -check-prefix=CHECK-EXPLICIT-NOPCH -input-file=%t<br>
+// CHECK-EXPLICIT-NOPCH: -dependency-file<br>
+// CHECK-EXPLICIT-NOPCH-NOT: -module-file-deps<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
</blockquote></div>