[llvm-commits] [llvm] r105675 - /llvm/trunk/include/llvm/ADT/DAGDeltaAlgorithm.h

Rafael Espindola rafael.espindola at gmail.com
Tue Jun 8 20:00:05 PDT 2010


Author: rafael
Date: Tue Jun  8 22:00:05 2010
New Revision: 105675

URL: http://llvm.org/viewvc/llvm-project?rev=105675&view=rev
Log:
Fix a gcc warning:

'class llvm::DAGDeltaAlgorithm' has virtual functions and accessible non-virtual destructor

Not sure if this is the best solution, but this class has state and some of the
classes that inherit from it also do, so it looks appropriate.

Modified:
    llvm/trunk/include/llvm/ADT/DAGDeltaAlgorithm.h

Modified: llvm/trunk/include/llvm/ADT/DAGDeltaAlgorithm.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ADT/DAGDeltaAlgorithm.h?rev=105675&r1=105674&r2=105675&view=diff
==============================================================================
--- llvm/trunk/include/llvm/ADT/DAGDeltaAlgorithm.h (original)
+++ llvm/trunk/include/llvm/ADT/DAGDeltaAlgorithm.h Tue Jun  8 22:00:05 2010
@@ -45,6 +45,8 @@
   typedef std::vector<changeset_ty> changesetlist_ty;
 
 public:
+  virtual ~DAGDeltaAlgorithm() {}
+
   /// Run - Minimize the DAG formed by the \arg Changes vertices and the \arg
   /// Dependencies edges by executing \see ExecuteOneTest() on subsets of
   /// changes and returning the smallest set which still satisfies the test





More information about the llvm-commits mailing list