[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