[PATCH] [RFC] Introduce support for multiple sanitizer blacklists (Clang side).

Alexey Samsonov vonosmas at gmail.com
Wed Jul 16 16:16:19 PDT 2014


================
Comment at: include/clang/Driver/SanitizerArgs.h:12
@@ -11,2 +11,3 @@
 
+#include "llvm/ADT/StringRef.h"
 #include "llvm/Option/Arg.h"
----------------
You may include clang/Basic/LLVM.h and use plain StringRef instead of llvm::StringRef.

================
Comment at: lib/CodeGen/BackendUtil.cpp:592
@@ -589,1 +591,3 @@
 
+std::unique_ptr<SpecialCaseList> clang::CreateSpecialCaseList(const CodeGenOptions &CGOpts) {
+  auto SCL = llvm::make_unique<SpecialCaseList>();
----------------
See my comment in LLVM part - this file doesn't look like the right place for this function.

================
Comment at: lib/Driver/SanitizerArgs.cpp:128
@@ +127,3 @@
+    Arg->claim();
+    if (llvm::sys::fs::exists(Arg->getValue())) {
+      addBlacklistFile(D, Arg->getValue());
----------------
nit: remove {} for if- and else- clauses for consistency.

http://reviews.llvm.org/D4547






More information about the cfe-commits mailing list