From: Rafael Espindola Date: Tue, 26 May 2015 00:32:28 +0000 (+0000) Subject: Call the destructors of the MCSections. X-Git-Tag: llvmorg-3.7.0-rc1~3849 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=24d285d31ba07be9edff7788aa5003192dbbfb3a;p=platform%2Fupstream%2Fllvm.git Call the destructors of the MCSections. They now contain the fragments. If we don't call the destructors the fragments leak. Found by an asan bot. llvm-svn: 238161 --- diff --git a/llvm/lib/MC/MCContext.cpp b/llvm/lib/MC/MCContext.cpp index 968fd26..0a453cd 100644 --- a/llvm/lib/MC/MCContext.cpp +++ b/llvm/lib/MC/MCContext.cpp @@ -10,6 +10,7 @@ #include "llvm/MC/MCContext.h" #include "llvm/ADT/SmallString.h" #include "llvm/ADT/Twine.h" +#include "llvm/MC/MCAssembler.h" #include "llvm/MC/MCAsmInfo.h" #include "llvm/MC/MCDwarf.h" #include "llvm/MC/MCLabel.h" @@ -54,6 +55,13 @@ MCContext::MCContext(const MCAsmInfo *mai, const MCRegisterInfo *mri, } MCContext::~MCContext() { + // Call the destructors so the fragments are freed + for (auto &I : ELFUniquingMap) + I.second->~MCSectionELF(); + for (auto &I : COFFUniquingMap) + I.second->~MCSectionCOFF(); + for (auto &I : MachOUniquingMap) + I.second->~MCSectionMachO(); if (AutoReset) reset();