pcc added a comment.

We should also make blacklists appear in the `--show-includes` output.


================
Comment at: lib/Frontend/DependencyFile.cpp:416-420
@@ -413,2 +415,7 @@
 
+  // Add extra dependencies to the end of the list.
+  for (auto ExtraDep : ExtraDeps) {
+    AddFilename(ExtraDep);
+  }
+
   // Write out the dependency targets, trying to avoid overly long
----------------
 If you move this code to the constructor you won't need to add an extra data 
member to this class.

================
Comment at: test/Frontend/dependency-gen.c:26
@@ -23,1 +25,3 @@
+// CHECK-SEVEN: {{ }}x.h
+// CHECK-SEVEN: .blacklist
 
----------------
Can we also test that the default blacklist appears in the `-MD` output if 
appropriate?


http://reviews.llvm.org/D11968



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

Reply via email to