Merge pull request #26 from WebAssembly/virtual-dtor

Add missing virtual dtor

Author: kripken

CMakeLists.txt

@@ -7,7 +7,7 @@ SET(CMAKE_RUNTIME_OUTPUT_DIRECTORY "${PROJECT_BINARY_DIR}/bin")
 
 SET(CMAKE_CXX_FLAGS "-std=c++11 ${CMAKE_CXX_FLAGS}")
 SET(CMAKE_CXX_FLAGS "-msse2 -mfpmath=sse ${CMAKE_CXX_FLAGS}")
-SET(CMAKE_CXX_FLAGS "-O2 -Wall ${CMAKE_CXX_FLAGS}")
+SET(CMAKE_CXX_FLAGS "-O2 ${CMAKE_CXX_FLAGS}")
 
 # clang doesn't print colored diagnostics when invoked from Ninja
 IF (UNIX AND

src/pass.cpp

@@ -86,7 +86,7 @@ P* PassRunner::getLast() {
 
 PassRunner::~PassRunner() {
   for (auto pass : passes) {
-    delete(pass);
+    delete pass;
   }
 }
 

src/wasm.h

@@ -1107,6 +1107,7 @@ class Module {
 
 template<class ReturnType>
 struct WasmVisitor {
+  virtual ~WasmVisitor() {}
   // should be pure virtual, but https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51048
   // Expression visitors
   virtual ReturnType visitBlock(Block *curr) { abort(); }