[Perf] (Espryt): a clean framebuffer attachment skips its per-draw resync behind the unit list's gate (rd12 monolith 74.5 -> 77.0 fps).

2026-11-07 01:53:24 +09:00 · 2026-10-07 10:50:37 -04:00
parent 0b0089e599
commit b556ec3134
1 changed files with 8 additions and 0 deletions
@@ -2325,6 +2325,14 @@ namespace MobileGL::MG_Backend::DirectGLES {
                                                 Vector<FboAttachmentSyncEntry>& list, Bool listValid) {
            if (listValid) {
                for (const auto& entry : list) {
+                    // P14: the unit list's aggregate clean gate first (SyncNeccessaryTextures): an
+                    // attachment whose storage, parameters and sampler are already the driver's
+                    // costs two record reads instead of three nested syncs and their SharedPtr
+                    // traffic. Same borrow rule as there - the list's key proves the twin alive.
+                    const auto* record = PipeTextureRecordForHandle(entry.Res);
+                    if (record != nullptr && entry.backend != nullptr &&
+                        entry.backend->IsDrawSyncCleanByRecord(entry.Res, *record))
+                        continue;
                    TextureImpl::SyncTextureToBackendByHandle(entry.Res, /*imageBindableStorageRequired=*/false);
                }
                return;