Add a default virtual dtor to Pass and check for missing virtual dtors

This commit is contained in:
Jean-Francois Roy 2017-02-28 11:34:18 -08:00 committed by David Neto
parent 0e9c24fdd1
commit 6a2b514826
2 changed files with 4 additions and 1 deletions

View File

@ -50,7 +50,7 @@ if(("${CMAKE_CXX_COMPILER_ID}" MATCHES "GNU") OR ("${CMAKE_CXX_COMPILER_ID}" MAT
set(COMPILER_IS_LIKE_GNU TRUE)
endif()
if(${COMPILER_IS_LIKE_GNU})
set(SPIRV_WARNINGS -Wall -Wextra -Wno-missing-field-initializers)
set(SPIRV_WARNINGS -Wall -Wextra -Wnon-virtual-dtor -Wno-missing-field-initializers)
option(SPIRV_WARN_EVERYTHING "Enable -Weverything" ${SPIRV_WARN_EVERYTHING})
if(${SPIRV_WARN_EVERYTHING})

View File

@ -44,6 +44,9 @@ class Pass {
// one if messages are of concern.
Pass() : consumer_(nullptr) {}
// Destructs the pass.
virtual ~Pass() = default;
// Returns a descriptive name for this pass.
virtual const char* name() const = 0;