Fix -Wnon-virtual-dtor warnings and renable -Werror

This adds a destructor to TreeVisitor to fix the following warning:

    'TreeVisitor' has virtual functions but non-virtual destructor [-Wnon-virtual-dtor]

Author: jonas

bindgen/CMakeLists.txt

@@ -46,6 +46,7 @@ execute_process(COMMAND ${LLVM_CONFIG_PROGRAM} --system-libs --link-static
 include_directories(SYSTEM ${LLVM_INCLUDE_DIR})
 
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${LLVM_CXX_FLAGS}")
+add_compile_options(-fexceptions -std=c++11 -Wall -Wconversion -Werror)
 
 if (${CMAKE_SYSTEM_NAME} MATCHES "Darwin")
   # macOS does not guarantee backwards compatible system calls and therefore
@@ -57,8 +58,6 @@ else()
   set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} ${LLVM_LINKER_FLAGS} -static")
 endif()
 
-add_compile_options(-Wall -Wconversion)
-
 add_executable(bindgen
   Main.cpp
   visitor/ScalaFrontendAction.h

bindgen/visitor/TreeVisitor.h

@@ -26,6 +26,7 @@ class TreeVisitor : public clang::RecursiveASTVisitor<TreeVisitor> {
     TreeVisitor(clang::CompilerInstance *CI, IR &ir)
         : astContext(&(CI->getASTContext())), typeTranslator(astContext, ir),
           cycleDetection(typeTranslator), ir(ir) {}
+    virtual ~TreeVisitor() {}
 
     virtual bool VisitFunctionDecl(clang::FunctionDecl *func);
     virtual bool VisitTypedefDecl(clang::TypedefDecl *tpdef);