diff options
author | 2018-06-18 12:52:47 -0400 | |
---|---|---|
committer | 2018-06-18 17:13:58 +0000 | |
commit | 19c1233c447f625c2522e7ecd0a0adecc629bb2f (patch) | |
tree | 891e06143986ade3aae8d4176ec8b426f70b896f /src/gpu/GrGeometryProcessor.h | |
parent | 146cf3ce7935019ecc63ce9e93450a8c122880d8 (diff) |
Change how vertex/instance attributes are handled in geometry processors.
* No longer register vertex/instance attributes on base class, just counts
* Separate instance and vertex attributes and remove InputRate and offset
* Make attributes constexpr where possible
Change-Id: I1f1d5e772fa177a96d2aeb805aab7b69f35bfae6
Reviewed-on: https://skia-review.googlesource.com/132405
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Chris Dalton <csmartdalton@google.com>
Diffstat (limited to 'src/gpu/GrGeometryProcessor.h')
-rw-r--r-- | src/gpu/GrGeometryProcessor.h | 37 |
1 files changed, 37 insertions, 0 deletions
diff --git a/src/gpu/GrGeometryProcessor.h b/src/gpu/GrGeometryProcessor.h index a12ab0058d..5035ad0f53 100644 --- a/src/gpu/GrGeometryProcessor.h +++ b/src/gpu/GrGeometryProcessor.h @@ -39,7 +39,44 @@ protected: fSampleShading = sampleShading; } + /** + * Recursive helpers for implementing onVertexAttribute or onInstanceAttribute. + */ + + template <typename... Args> + static const Attribute& IthAttribute(int i, const Attribute& attr0, const Args&... attrs) { + SkASSERT(attr0.isInitialized()); + return (0 == i) ? attr0 : IthAttribute(i - 1, attrs...); + } + + static const Attribute& IthAttribute(int i) { + SK_ABORT("Illegal attribute Index"); + static constexpr Attribute kBogus; + return kBogus; + } + + template <typename... Args> + static const Attribute& IthInitializedAttribute(int i, const Attribute& attr0, + const Args&... attrs) { + if (attr0.isInitialized()) { + if (0 == i) { + return attr0; + } + i -= 1; + } + return IthInitializedAttribute(i, attrs...); + } + + static const Attribute& IthInitializedAttribute(int i) { return IthAttribute(i); } + private: + // Since most subclasses don't use instancing provide a default implementation for that case. + const Attribute& onInstanceAttribute(int i) const override { + SK_ABORT("No instanced attributes"); + static constexpr Attribute kBogus; + return kBogus; + } + bool fWillUseGeoShader; float fSampleShading; |