commit 56a1f06db241fea9fd1e7d1367fa56d94d5bf3d7
author Mike Klein <mtklein@google.com> Sun Feb 28 15:21:08 2021 +0000
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Sun Feb 28 15:21:17 2021 +0000
tree dd866ea5fd2d0bc763ceba3ff0bd1ece34021129
parent 59b389a407dede4fe7cf54ebd52e93363a5b52da

Revert "Reland "Remove GrProgramDesc::KeyHeader structure""

This reverts commit 202420e0147b02f963de761a903b2c430dfa0cec.

Reason for revert: MSVC build failures

Original change's description:
> Reland "Remove GrProgramDesc::KeyHeader structure"
>
> This is a reland of 4bcd58afbf1a1993e453231c575b47a0ae1ec24d
>
> Original change's description:
> > Remove GrProgramDesc::KeyHeader structure
> >
> > Instead, just fold all of this information into the key, like everything
> > else. The only value that was accessed elsewhere is the initial key
> > length. That doesn't need to be part of the key, so store it separately
> > in the GrProgramDesc.
> >
> > Removing this special case logic is just the first step in revising how
> > we assemble keys.
> >
> > Bug: skia:11372
> > Change-Id: I52eb76812045e1906797cb37e809cfd0b3332ef0
> > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/376797
> > Reviewed-by: Brian Salomon <bsalomon@google.com>
> > Commit-Queue: Brian Osman <brianosman@google.com>
>
> Bug: skia:11372
> Change-Id: I2cdb49aee3537e54dad9af1f9b47cf1aed1aca21
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/376849
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Commit-Queue: Brian Osman <brianosman@google.com>

TBR=bsalomon@google.com,robertphillips@google.com,brianosman@google.com

Change-Id: I0f4a76fa5d38d516554eed642eb9ab5f3222d9f3
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: skia:11372
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/377231
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>

src/gpu/GrProgramDesc.h

diff --git a/src/gpu/GrProgramDesc.h b/src/gpu/GrProgramDesc.h
index 7eaf880..011edfd 100644
--- a/src/gpu/GrProgramDesc.h
+++ b/src/gpu/GrProgramDesc.h
@@ -22,7 +22,7 @@
  */
 class GrProgramDesc {
 public:
-    GrProgramDesc(const GrProgramDesc& other) = default;
+    GrProgramDesc(const GrProgramDesc& other) : fKey(other.fKey) {}   // for SkLRUCache
 
     bool isValid() const { return !fKey.empty(); }
 
@@ -41,7 +41,6 @@
         uint32_t keyLength = other.keyLength();
         fKey.reset(SkToInt(keyLength));
         memcpy(fKey.begin(), other.fKey.begin(), keyLength);
-        fInitialKeyLength = other.fInitialKeyLength;
         return *this;
     }
 
@@ -66,7 +65,7 @@
         return !(*this == other);
     }
 
-    uint32_t initialKeyLength() const { return fInitialKeyLength; }
+    uint32_t initialKeyLength() const { return this->header().fInitialKeyLength; }
 
 protected:
     friend class GrDawnCaps;
@@ -101,11 +100,48 @@
         return true;
     }
 
+    // TODO: this should be removed and converted to just data added to the key
+    struct KeyHeader {
+        // Set to uniquely identify any swizzling of the shader's output color(s).
+        uint16_t fWriteSwizzle;
+        uint8_t fColorFragmentProcessorCnt; // Can be packed into 4 bits if required.
+        uint8_t fCoverageFragmentProcessorCnt;
+        // Set to uniquely identify the rt's origin, or 0 if the shader does not require this info.
+        uint32_t fSurfaceOriginKey : 2;
+        uint32_t fProcessorFeatures : 1;
+        uint32_t fSnapVerticesToPixelCenters : 1;
+        uint32_t fHasPointSize : 1;
+        // This is the key size (in bytes) after core key construction. It doesn't include any
+        // portions added by the platform-specific backends.
+        uint32_t fInitialKeyLength : 27;
+    };
+    static_assert(sizeof(KeyHeader) == 8);
+
+    const KeyHeader& header() const { return *this->atOffset<KeyHeader, kHeaderOffset>(); }
+
+    template<typename T, size_t OFFSET> T* atOffset() {
+        return reinterpret_cast<T*>(reinterpret_cast<intptr_t>(fKey.begin()) + OFFSET);
+    }
+
+    template<typename T, size_t OFFSET> const T* atOffset() const {
+        return reinterpret_cast<const T*>(reinterpret_cast<intptr_t>(fKey.begin()) + OFFSET);
+    }
+
+    // The key, stored in fKey, is composed of two parts:
+    // 1. Header struct defined above.
+    // 2. A Backend specific payload which includes the per-processor keys.
+    enum KeyOffsets {
+        kHeaderOffset = 0,
+        kHeaderSize = SkAlign4(sizeof(KeyHeader)),
+        // This is the offset into the backend-specific part of the key, which includes
+        // per-processor keys.
+        kProcessorKeysOffset = kHeaderOffset + kHeaderSize,
+    };
+
     enum {
-        kHeaderSize            = 4,    // "header" in ::Build
         kMaxPreallocProcessors = 8,
         kIntsPerProcessor      = 4,    // This is an overestimate of the average effect key size.
-        kPreAllocSize = kHeaderSize +
+        kPreAllocSize = kHeaderOffset + kHeaderSize +
                         kMaxPreallocProcessors * sizeof(uint32_t) * kIntsPerProcessor,
     };
 
@@ -113,7 +149,6 @@
 
 private:
     SkSTArray<kPreAllocSize, uint8_t, true> fKey;
-    uint32_t fInitialKeyLength = 0;
 };
 
 #endif