[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