From 025983a8dd27fd5ff1defa11ebcd88ecd7248773 Mon Sep 17 00:00:00 2001 From: David Blaikie Date: Wed, 9 May 2012 18:31:50 +0000 Subject: [PATCH] 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 --- include/clang/Tooling/ArgumentsAdjusters.h | 3 +++ lib/Tooling/ArgumentsAdjusters.cpp | 3 +++ 2 files changed, 6 insertions(+) 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 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) { -- 2.40.0