summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Target/Sparc/SparcTargetMachine.h
diff options
context:
space:
mode:
authorReid Kleckner <reid@kleckner.net>2014-11-20 23:37:18 +0000
committerReid Kleckner <reid@kleckner.net>2014-11-20 23:37:18 +0000
commit357600eab56533a9b1a097b7675b3888cd50365d (patch)
tree764c606f71a8cd161bd0a3d4d2cb7b13848f27e3 /llvm/lib/Target/Sparc/SparcTargetMachine.h
parent661f29dde4475f21a9d94193ace2ad26aac14001 (diff)
downloadbcm5719-llvm-357600eab56533a9b1a097b7675b3888cd50365d.tar.gz
bcm5719-llvm-357600eab56533a9b1a097b7675b3888cd50365d.zip
Add out of line virtual destructors to all LLVMTargetMachine subclasses
These recently all grew a unique_ptr<TargetLoweringObjectFile> member in r221878. When anyone calls a virtual method of a class, clang-cl requires all virtual methods to be semantically valid. This includes the implicit virtual destructor, which triggers instantiation of the unique_ptr destructor, which fails because the type being deleted is incomplete. This is just part of the ongoing saga of PR20337, which is affecting Blink as well. Because the MSVC ABI doesn't have key functions, we end up referencing the vtable and implicit destructor on any virtual call through a class. We don't actually end up emitting the dtor, so it'd be good if we could avoid this unneeded type completion work. llvm-svn: 222480
Diffstat (limited to 'llvm/lib/Target/Sparc/SparcTargetMachine.h')
-rw-r--r--llvm/lib/Target/Sparc/SparcTargetMachine.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Target/Sparc/SparcTargetMachine.h b/llvm/lib/Target/Sparc/SparcTargetMachine.h
index f73c5c527cf..096e7c8485a 100644
--- a/llvm/lib/Target/Sparc/SparcTargetMachine.h
+++ b/llvm/lib/Target/Sparc/SparcTargetMachine.h
@@ -28,6 +28,7 @@ public:
StringRef CPU, StringRef FS, const TargetOptions &Options,
Reloc::Model RM, CodeModel::Model CM,
CodeGenOpt::Level OL, bool is64bit);
+ ~SparcTargetMachine() override;
const SparcSubtarget *getSubtargetImpl() const override { return &Subtarget; }
OpenPOWER on IntegriCloud