pica: Switch to dirty flags

Author: raphaelthegreat

Diff for: src/video_core/debug_utils/debug_utils.cpp

@@ -272,11 +272,10 @@ void StartPicaTracing() {
 }
 
 void OnPicaRegWrite(u16 cmd_id, u16 mask, u32 value) {
-    std::lock_guard lock(pica_trace_mutex);
-
     if (!g_is_pica_tracing)
         return;
 
+    std::lock_guard lock(pica_trace_mutex);
     pica_trace->writes.push_back(PicaTrace::Write{cmd_id, mask, value});
 }
 

Diff for: src/video_core/pica/pica_core.cpp

@@ -412,8 +412,9 @@ void PicaCore::WriteInternalReg(u32 id, u32 value, u32 mask) {
         break;
     }
 
-    // Notify the rasterizer an internal register was updated.
-    rasterizer->NotifyPicaRegisterChanged(id);
+    // Mark register as dirty
+    dirty_flags.set(id);
+    LOG_INFO(HW_GPU, "Setting dirty flag for reg = {:#x}", id);
 }
 
 void PicaCore::SubmitImmediate(u32 value) {

Diff for: src/video_core/pica/pica_core.h

@@ -4,6 +4,7 @@
 
 #pragma once
 
+#include <bitset>
 #include "core/hle/service/gsp/gsp_interrupt.h"
 #include "video_core/pica/geometry_pipeline.h"
 #include "video_core/pica/packed_attribute.h"
@@ -240,8 +241,11 @@ class PicaCore {
         }
     };
 
+    using DirtyFlags = std::bitset<Regs::NUM_REGS>;
+
     RegsLcd regs_lcd{};
     Regs regs{};
+    DirtyFlags dirty_flags{};
     // TODO: Move these to a separate shader scheduler class
     GeometryShaderUnit gs_unit;
     ShaderSetup vs_setup;