[cfe-commits] [PATCH] Clang support for sanitizer blacklists.

[Alexey Samsonov]

  r169144, thanks!


================
Comment at: include/clang/Driver/Options.td:397-398
@@ +396,4 @@
+                          HelpText<"Path to blacklist file for sanitizers">;
+def fno_sanitize_blacklist : Flag<["-"], "fno-sanitize-blacklist">,
+                             Group<f_clang_Group>, Flags<[CC1Option]>,
+                             HelpText<"Don't use blacklist file for sanitizers">;
----------------
Richard Smith wrote:
> I think this doesn't need to be a CC1Option.
Done

================
Comment at: lib/Driver/Tools.cpp:1485
@@ +1484,3 @@
+
+  // Parse -f(no)sanitize-blacklist options.
+  if (Arg *BLArg = Args.getLastArg(options::OPT_fsanitize_blacklist,
----------------
Richard Smith wrote:
> -f(no-)sanitize-blacklist? :)
Done

================
Comment at: test/Driver/fsanitize-blacklist.c:2-3
@@ +1,4 @@
+// General blacklist usage.
+// RUN: %clang -fsanitize=address -fsanitize-blacklist=%s %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-BLACKLIST
+// CHECK-BLACKLIST: -fsanitize-blacklist
+
----------------
Richard Smith wrote:
> It'd be good to have a simple test that the flag is being correctly passed to the ASan pass (maybe just CHECK-NOT: asan in the generated IR?)
Do you mean creating a test together with minimalistic blacklist and checking that IR for blacklisted functions doesn't have ASan instrumentation (in addition to ASan output tests)?


http://llvm-reviews.chandlerc.com/D152

BRANCH
  svn

ARCANIST PROJECT
  clang