[PATCH] D11968: Add sanitizer blacklists to the rules generated with -M/-MM/-MD/-MMD.
Peter Collingbourne via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 11 17:35:24 PDT 2015
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
More information about the cfe-commits
mailing list