Reviewed-by: Samuel Pitoiset <samuel.pitoi...@gmail.com>

On 05/31/2017 03:05 AM, Ian Romanick wrote:
This patch is

Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>

On 05/30/2017 01:47 PM, Rob Herring wrote:
Make ast_node destructor virtual to fix the following clang warning:

src/compiler/glsl/ast.h:51:4: warning: destructor called on non-final 
'ast_node' that has virtual functions but non-virtual destructor 
[-Wdelete-non-virtual-dtor]

Signed-off-by: Rob Herring <r...@kernel.org>
---
  src/compiler/glsl/ast.h | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/src/compiler/glsl/ast.h b/src/compiler/glsl/ast.h
index 9327e0397935..5515dabf0504 100644
--- a/src/compiler/glsl/ast.h
+++ b/src/compiler/glsl/ast.h
@@ -50,6 +50,8 @@ class ast_node {
  public:
     DECLARE_LINEAR_ZALLOC_CXX_OPERATORS(ast_node);
+ virtual ~ast_node() {};
+
     /**
      * Print an AST node in something approximating the original GLSL code
      */


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to