This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Fix memory leak on WasmEHFuncInfo
ClosedPublic

Authored by aheejin on Sep 26 2018, 5:33 PM.

Reviewers

Commits

Summary

WasmEHFuncInfo objects were not being properly deleted.

Repository

rL LLVM

Build Status

Buildable 23173
Build 23172: arc lint + arc unit

This revision is now accepted and ready to land.Sep 27 2018, 8:44 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

lib/

CodeGen/

5 lines

Show First 20 Lines • Show All 240 Lines • ▼ Show 20 Lines	if (JumpTableInfo) {
JumpTableInfo->~MachineJumpTableInfo();		JumpTableInfo->~MachineJumpTableInfo();
Allocator.Deallocate(JumpTableInfo);		Allocator.Deallocate(JumpTableInfo);
}		}

if (WinEHInfo) {		if (WinEHInfo) {
WinEHInfo->~WinEHFuncInfo();		WinEHInfo->~WinEHFuncInfo();
Allocator.Deallocate(WinEHInfo);		Allocator.Deallocate(WinEHInfo);
}		}

		if (WasmEHInfo) {
		WasmEHInfo->~WasmEHFuncInfo();
		Allocator.Deallocate(WasmEHInfo);
		}
}		}

const DataLayout &MachineFunction::getDataLayout() const {		const DataLayout &MachineFunction::getDataLayout() const {
return F.getParent()->getDataLayout();		return F.getParent()->getDataLayout();
}		}

/// Get the JumpTableInfo for this function.		/// Get the JumpTableInfo for this function.
/// If it does not already exist, allocate one.		/// If it does not already exist, allocate one.
▲ Show 20 Lines • Show All 794 Lines • Show Last 20 Lines