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