[dxvk] Simplify pipeline instance data

src/dxvk/dxvk_compute.cpp

@@ -30,7 +30,7 @@ namespace dxvk {
   
   DxvkComputePipeline::~DxvkComputePipeline() {
     for (const auto& instance : m_pipelines)
-      this->destroyPipeline(instance.pipeline());
+      this->destroyPipeline(instance.handle);
   }
   
   
@@ -60,7 +60,7 @@ namespace dxvk {
         }
       }
 
-      return instance->pipeline();
+      return instance->handle;
     }
   }
 
@@ -88,7 +88,7 @@ namespace dxvk {
   DxvkComputePipelineInstance* DxvkComputePipeline::findInstance(
     const DxvkComputePipelineStateInfo& state) {
     for (auto& instance : m_pipelines) {
-      if (instance.isCompatible(state))
+      if (instance.state == state)
         return &instance;
     }
     

src/dxvk/dxvk_compute.h

@@ -39,44 +39,15 @@ namespace dxvk {
   /**
    * \brief Compute pipeline instance
    */
-  class DxvkComputePipelineInstance {
+  struct DxvkComputePipelineInstance {
-
+    DxvkComputePipelineInstance() { }
-  public:
-
-    DxvkComputePipelineInstance()
-    : m_stateVector (),
-      m_pipeline    (VK_NULL_HANDLE) { }
-
     DxvkComputePipelineInstance(
-      const DxvkComputePipelineStateInfo& state,
+      const DxvkComputePipelineStateInfo& state_,
-            VkPipeline                    pipe)
+            VkPipeline                    handle_)
-    : m_stateVector (state),
+    : state(state_), handle(handle_) { }
-      m_pipeline    (pipe) { }
-
-    /**
-     * \brief Checks for matching pipeline state
-     * 
-     * \param [in] stateVector Graphics pipeline state
-     * \param [in] renderPass Render pass handle
-     * \returns \c true if the specialization is compatible
-     */
-    bool isCompatible(const DxvkComputePipelineStateInfo& state) const {
-      return m_stateVector == state;
-    }
-
-    /**
-     * \brief Retrieves pipeline
-     * \returns The pipeline handle
-     */
-    VkPipeline pipeline() const {
-      return m_pipeline;
-    }
-
-  private:
-
-    DxvkComputePipelineStateInfo m_stateVector;
-    VkPipeline                   m_pipeline;
 
+    DxvkComputePipelineStateInfo state;
+    VkPipeline                   handle = VK_NULL_HANDLE;
   };
   
   

src/dxvk/dxvk_graphics.cpp

@@ -493,7 +493,7 @@ namespace dxvk {
   
   DxvkGraphicsPipeline::~DxvkGraphicsPipeline() {
     for (const auto& instance : m_pipelines)
-      this->destroyPipeline(instance.pipeline());
+      this->destroyPipeline(instance.handle);
   }
   
   
@@ -545,7 +545,7 @@ namespace dxvk {
       }
     }
 
-    return instance->pipeline();
+    return instance->handle;
   }
 
 
@@ -576,7 +576,7 @@ namespace dxvk {
   DxvkGraphicsPipelineInstance* DxvkGraphicsPipeline::findInstance(
     const DxvkGraphicsPipelineStateInfo& state) {
     for (auto& instance : m_pipelines) {
-      if (instance.isCompatible(state))
+      if (instance.state == state)
         return &instance;
     }
     

src/dxvk/dxvk_graphics.h

@@ -236,44 +236,15 @@ namespace dxvk {
    * Stores a state vector and the
    * corresponding pipeline handle.
    */
-  class DxvkGraphicsPipelineInstance {
+  struct DxvkGraphicsPipelineInstance {
-
+    DxvkGraphicsPipelineInstance() { }
-  public:
-
-    DxvkGraphicsPipelineInstance()
-    : m_stateVector (),
-      m_pipeline    (VK_NULL_HANDLE) { }
-
     DxvkGraphicsPipelineInstance(
-      const DxvkGraphicsPipelineStateInfo&  state,
+      const DxvkGraphicsPipelineStateInfo&  state_,
-            VkPipeline                      pipe)
+            VkPipeline                      handle_)
-    : m_stateVector (state),
+    : state(state_), handle(handle_) { }
-      m_pipeline    (pipe) { }
-
-    /**
-     * \brief Checks for matching pipeline state
-     * 
-     * \param [in] stateVector Graphics pipeline state
-     * \returns \c true if the specialization is compatible
-     */
-    bool isCompatible(
-      const DxvkGraphicsPipelineStateInfo&  state) {
-      return m_stateVector == state;
-    }
-
-    /**
-     * \brief Retrieves pipeline
-     * \returns The pipeline handle
-     */
-    VkPipeline pipeline() const {
-      return m_pipeline;
-    }
-
-  private:
-
-    DxvkGraphicsPipelineStateInfo m_stateVector;
-    VkPipeline                    m_pipeline;
 
+    DxvkGraphicsPipelineStateInfo state;
+    VkPipeline                    handle = VK_NULL_HANDLE;
   };