diff options
author | Reid Kleckner <reid@kleckner.net> | 2014-11-20 23:37:18 +0000 |
---|---|---|
committer | Reid Kleckner <reid@kleckner.net> | 2014-11-20 23:37:18 +0000 |
commit | 357600eab56533a9b1a097b7675b3888cd50365d (patch) | |
tree | 764c606f71a8cd161bd0a3d4d2cb7b13848f27e3 /llvm/lib/Target/MSP430/MSP430TargetMachine.h | |
parent | 661f29dde4475f21a9d94193ace2ad26aac14001 (diff) | |
download | llvm-357600eab56533a9b1a097b7675b3888cd50365d.zip llvm-357600eab56533a9b1a097b7675b3888cd50365d.tar.gz llvm-357600eab56533a9b1a097b7675b3888cd50365d.tar.bz2 |
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/MSP430/MSP430TargetMachine.h')
-rw-r--r-- | llvm/lib/Target/MSP430/MSP430TargetMachine.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Target/MSP430/MSP430TargetMachine.h b/llvm/lib/Target/MSP430/MSP430TargetMachine.h index abe0643..0e54ed6 100644 --- a/llvm/lib/Target/MSP430/MSP430TargetMachine.h +++ b/llvm/lib/Target/MSP430/MSP430TargetMachine.h @@ -32,6 +32,7 @@ public: StringRef CPU, StringRef FS, const TargetOptions &Options, Reloc::Model RM, CodeModel::Model CM, CodeGenOpt::Level OL); + ~MSP430TargetMachine() override; const MSP430Subtarget *getSubtargetImpl() const override { return &Subtarget; |