Skip to content

Commit

Permalink
OpChain has subclasses, so add a virtual destructor.
Browse files Browse the repository at this point in the history
Summary:
OpChain has subclasses, so add a virtual destructor.

This fixes an issue when deleting subclasses of OpChain (see MatchSMLAD() specifically) in r337701.

Reviewers: javed.absar

Subscribers: llvm-commits, SjoerdMeijer, samparker

Differential Revision: https://reviews.llvm.org/D49681

llvm-svn: 337713
  • Loading branch information
rupprecht committed Jul 23, 2018
1 parent 966b520 commit e5daf61
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions llvm/lib/Target/ARM/ARMParallelDSP.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ namespace {
bool ReadOnly = true;

OpChain(Instruction *I, ValueList &vl) : Root(I), AllValues(vl) { }
virtual ~OpChain() = default;

void SetMemoryLocations() {
const auto Size = MemoryLocation::UnknownSize;
Expand Down

0 comments on commit e5daf61

Please sign in to comment.