[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