#12: enable Werror for the plugin

DARMA-tasking · Feb 8, 2021 · a907606 · a907606
1 parent f899001
commit a907606
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -137,10 +137,15 @@ add_library(
 )
 
 target_include_directories(
-  sanitizer-plugin PUBLIC
+  sanitizer-plugin PRIVATE
   "${LLVM_INCLUDE_DIRS};${CLANG_INCLUDE_DIRS}"
 )
 
+target_compile_options(
+  sanitizer-plugin PRIVATE
+  -Wall -Werror
+)
+
 add_library(vt::lib::sanitizer_rt ALIAS sanitizer_rt)
 
 target_compile_definitions(

diff --git a/src/plugin/plugin.cc b/src/plugin/plugin.cc
@@ -64,7 +64,7 @@ void printWarning(FieldDecl const* field_decl) {
 
 void SanitizerMatcher::run(const MatchFinder::MatchResult &result) {
   auto record = result.Nodes.getNodeAs<CXXRecordDecl>("record");
-  auto method = result.Nodes.getNodeAs<FunctionTemplateDecl>("method");
+  // auto method = result.Nodes.getNodeAs<FunctionTemplateDecl>("method");
   auto binary_op = result.Nodes.getNodeAs<BinaryOperator>("binary");
 
   // llvm::errs() << "\nProcessing serialized fields [" << record->getQualifiedNameAsString() << "]\n";