6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2024-12-13 04:30:23 +00:00
Code Issues Projects Releases Wiki Activity

Reapply fix in r217988 (reverted in r217989) and remove the alternative fix committed in r217987.

Browse Source 
This type isn't owned polymorphically (as demonstrated by making the
dtor protected and everything still compiling) so just address the
warning by protecting the base dtor and making the derived class final.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@217990 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
David Blaikie 2014-09-17 22:27:36 +00:00
parent f8b9ec99b8
commit 52675f31e5
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
include/llvm/Transforms/Utils/SimplifyLibCalls.h
View File

@ -36,14 +36,13 @@ private:
const TargetLibraryInfo *TLI;
bool UnsafeFPShrink;
protected:
~LibCallSimplifier() {}
public:
LibCallSimplifier(const DataLayout *TD, const TargetLibraryInfo *TLI,
bool UnsafeFPShrink);
/// ~LibCallSimplifier - Adding the virtual destructor back in to satisfy
/// -Wnon-virtual-dtor. This class is used polymorphically by InstCombine.
virtual ~LibCallSimplifier() {}
/// optimizeCall - Take the given call instruction and return a more
/// optimal value to replace the instruction with or 0 if a more
/// optimal form can't be found. Note that the returned value may

2
lib/Transforms/InstCombine/InstructionCombining.cpp
View File

@ -2908,7 +2908,7 @@ bool InstCombiner::DoOneIteration(Function &F, unsigned Iteration) {
}
namespace {
class InstCombinerLibCallSimplifier : public LibCallSimplifier {
class InstCombinerLibCallSimplifier final : public LibCallSimplifier {
InstCombiner *IC;
public:
InstCombinerLibCallSimplifier(const DataLayout *DL,