From: David Blaikie <dblaikie@gmail.com>
Date: Wed, 9 May 2012 18:31:50 +0000 (+0000)
Subject: Fix -Wdelete-non-virtual-dtor in Tooling code.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=025983a8dd27fd5ff1defa11ebcd88ecd7248773;p=clang

Fix -Wdelete-non-virtual-dtor in Tooling code.

Also, add an anchor as per http://llvm.org/docs/CodingStandards.html#ll_virtual_anch

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@156495 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/include/clang/Tooling/ArgumentsAdjusters.h b/include/clang/Tooling/ArgumentsAdjusters.h
index 28d2a38a4c..492ddd2b00 100644
--- a/include/clang/Tooling/ArgumentsAdjusters.h
+++ b/include/clang/Tooling/ArgumentsAdjusters.h
@@ -32,6 +32,7 @@ typedef std::vector<std::string> CommandLineArguments;
 /// which is responsible for command line arguments modification before
 /// the arguments are used to run a frontend action.
 class ArgumentsAdjuster {
+  virtual void anchor();
 public:
   /// \brief Returns adjusted command line arguments.
   ///
@@ -39,6 +40,8 @@ public:
   ///
   /// \returns Modified sequence of command line arguments.
   virtual CommandLineArguments Adjust(const CommandLineArguments &Args) = 0;
+  virtual ~ArgumentsAdjuster() {
+  }
 };
 
 /// \brief Syntax check only command line adjuster.
diff --git a/lib/Tooling/ArgumentsAdjusters.cpp b/lib/Tooling/ArgumentsAdjusters.cpp
index 73ad39816b..31dd465995 100644
--- a/lib/Tooling/ArgumentsAdjusters.cpp
+++ b/lib/Tooling/ArgumentsAdjusters.cpp
@@ -17,6 +17,9 @@
 namespace clang {
 namespace tooling {
 
+void ArgumentsAdjuster::anchor() {
+}
+
 /// Add -fsyntax-only option to the commnand line arguments.
 CommandLineArguments
 ClangSyntaxOnlyAdjuster::Adjust(const CommandLineArguments &Args) {