Browse Source

video_core/gpu: Make GPU's destructor virtual

Because of the recent separation of GPU functionality into sync/async
variants, we need to mark the destructor virtual to provide proper
destruction behavior, given we use the base class within the System
class.

Prior to this, it was undefined behavior whether or not the destructor
in the derived classes would ever execute.
nce_cpp
Lioncash 7 years ago
parent
commit
bd263e75cc
  1. 2
      src/video_core/gpu.h
  2. 2
      src/video_core/gpu_asynch.h
  3. 2
      src/video_core/gpu_synch.h

2
src/video_core/gpu.h

@ -123,7 +123,7 @@ class GPU {
public: public:
explicit GPU(Core::System& system, VideoCore::RendererBase& renderer); explicit GPU(Core::System& system, VideoCore::RendererBase& renderer);
~GPU();
virtual ~GPU();
struct MethodCall { struct MethodCall {
u32 method{}; u32 method{};

2
src/video_core/gpu_asynch.h

@ -21,7 +21,7 @@ class ThreadManager;
class GPUAsynch : public Tegra::GPU { class GPUAsynch : public Tegra::GPU {
public: public:
explicit GPUAsynch(Core::System& system, VideoCore::RendererBase& renderer); explicit GPUAsynch(Core::System& system, VideoCore::RendererBase& renderer);
~GPUAsynch();
~GPUAsynch() override;
void PushGPUEntries(Tegra::CommandList&& entries) override; void PushGPUEntries(Tegra::CommandList&& entries) override;
void SwapBuffers( void SwapBuffers(

2
src/video_core/gpu_synch.h

@ -16,7 +16,7 @@ namespace VideoCommon {
class GPUSynch : public Tegra::GPU { class GPUSynch : public Tegra::GPU {
public: public:
explicit GPUSynch(Core::System& system, VideoCore::RendererBase& renderer); explicit GPUSynch(Core::System& system, VideoCore::RendererBase& renderer);
~GPUSynch();
~GPUSynch() override;
void PushGPUEntries(Tegra::CommandList&& entries) override; void PushGPUEntries(Tegra::CommandList&& entries) override;
void SwapBuffers( void SwapBuffers(

Loading…
Cancel
Save