Merge pull request #8012 from stenzek/bounding-box-shaders

PixelShaderGen: Don't emit bounding box shader code for old UIDs when disabled

1 files changed, 20 insertions, 16 deletions

diff --git a/Source/Core/VideoCommon/ShaderCache.cpp b/Source/Core/VideoCommon/ShaderCache.cpp
index e1712ec8f3..3e4ed24662 100644
--- a/Source/Core/VideoCommon/ShaderCache.cpp
+++ b/Source/Core/VideoCommon/ShaderCache.cpp
@@ -641,17 +641,19 @@ const AbstractPipeline* ShaderCache::InsertGXPipeline(const GXPipelineUid& confi
   auto& entry = m_gx_pipeline_cache[config];
   entry.second = false;
   if (!entry.first && pipeline)
+  {
     entry.first = std::move(pipeline);
 
-  if (g_ActiveConfig.bShaderCache)
-  {
-    auto cache_data = entry.first->GetCacheData();
-    if (!cache_data.empty())
+    if (g_ActiveConfig.bShaderCache)
     {
-      SerializedGXPipelineUid disk_uid;
-      SerializePipelineUid(config, disk_uid);
-      m_gx_pipeline_disk_cache.Append(disk_uid, cache_data.data(),
-                                      static_cast<u32>(cache_data.size()));
+      auto cache_data = entry.first->GetCacheData();
+      if (!cache_data.empty())
+      {
+        SerializedGXPipelineUid disk_uid;
+        SerializePipelineUid(config, disk_uid);
+        m_gx_pipeline_disk_cache.Append(disk_uid, cache_data.data(),
+                                        static_cast<u32>(cache_data.size()));
+      }
     }
   }
 
@@ -665,17 +667,19 @@ ShaderCache::InsertGXUberPipeline(const GXUberPipelineUid& config,
   auto& entry = m_gx_uber_pipeline_cache[config];
   entry.second = false;
   if (!entry.first && pipeline)
+  {
     entry.first = std::move(pipeline);
 
-  if (g_ActiveConfig.bShaderCache)
-  {
-    auto cache_data = entry.first->GetCacheData();
-    if (!cache_data.empty())
+    if (g_ActiveConfig.bShaderCache)
     {
-      SerializedGXUberPipelineUid disk_uid;
-      SerializePipelineUid(config, disk_uid);
-      m_gx_uber_pipeline_disk_cache.Append(disk_uid, cache_data.data(),
-                                           static_cast<u32>(cache_data.size()));
+      auto cache_data = entry.first->GetCacheData();
+      if (!cache_data.empty())
+      {
+        SerializedGXUberPipelineUid disk_uid;
+        SerializePipelineUid(config, disk_uid);
+        m_gx_uber_pipeline_disk_cache.Append(disk_uid, cache_data.data(),
+                                             static_cast<u32>(cache_data.size()));
+      }
     }
   }