From fd12788967729dc3712c23723957ab6b15d36822 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Thu, 9 May 2019 19:04:41 -0400 Subject: [PATCH] video_core/memory_manager: Default the destructor within the cpp file Makes the class less surprising when it comes to forward declaring the type, and also prevents inlining the destruction code of the class, given it contains non-trivial types. --- src/video_core/memory_manager.cpp | 2 ++ src/video_core/memory_manager.h | 1 + 2 files changed, 3 insertions(+) diff --git a/src/video_core/memory_manager.cpp b/src/video_core/memory_manager.cpp index 6c98c6701..2890ea947 100644 --- a/src/video_core/memory_manager.cpp +++ b/src/video_core/memory_manager.cpp @@ -25,6 +25,8 @@ MemoryManager::MemoryManager(VideoCore::RasterizerInterface& rasterizer) : raste UpdatePageTableForVMA(initial_vma); } +MemoryManager::~MemoryManager() = default; + GPUVAddr MemoryManager::AllocateSpace(u64 size, u64 align) { const u64 aligned_size{Common::AlignUp(size, page_size)}; const GPUVAddr gpu_addr{FindFreeRegion(address_space_base, aligned_size)}; diff --git a/src/video_core/memory_manager.h b/src/video_core/memory_manager.h index ade18c139..4f7b57f8e 100644 --- a/src/video_core/memory_manager.h +++ b/src/video_core/memory_manager.h @@ -48,6 +48,7 @@ struct VirtualMemoryArea { class MemoryManager final { public: MemoryManager(VideoCore::RasterizerInterface& rasterizer); + ~MemoryManager(); GPUVAddr AllocateSpace(u64 size, u64 align); GPUVAddr AllocateSpace(GPUVAddr addr, u64 size, u64 align);