From 19c1233c447f625c2522e7ecd0a0adecc629bb2f Mon Sep 17 00:00:00 2001 From: Brian Salomon Date: Mon, 18 Jun 2018 12:52:47 -0400 Subject: 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 Reviewed-by: Chris Dalton --- tests/GrMeshTest.cpp | 51 +++++++++++++++++++++++++++++++-------------------- 1 file changed, 31 insertions(+), 20 deletions(-) (limited to 'tests/GrMeshTest.cpp') diff --git a/tests/GrMeshTest.cpp b/tests/GrMeshTest.cpp index 4535f13975..4ca0e1d10a 100644 --- a/tests/GrMeshTest.cpp +++ b/tests/GrMeshTest.cpp @@ -292,35 +292,46 @@ private: class GrMeshTestProcessor : public GrGeometryProcessor { public: GrMeshTestProcessor(bool instanced, bool hasVertexBuffer) - : INHERITED(kGrMeshTestProcessor_ClassID) - , fInstanceLocation(nullptr) - , fVertex(nullptr) - , fColor(nullptr) { + : INHERITED(kGrMeshTestProcessor_ClassID) { if (instanced) { - fInstanceLocation = &this->addInstanceAttrib("location", kHalf2_GrVertexAttribType); + fInstanceLocation = {"location", kHalf2_GrVertexAttribType}; + fColor = {"color", kUByte4_norm_GrVertexAttribType}; + this->setInstanceAttributeCnt(2); if (hasVertexBuffer) { - fVertex = &this->addVertexAttrib("vertex", kHalf2_GrVertexAttribType); + fVertex = {"vertex", kHalf2_GrVertexAttribType}; + this->setVertexAttributeCnt(1); } - fColor = &this->addInstanceAttrib("color", kUByte4_norm_GrVertexAttribType); } else { - fVertex = &this->addVertexAttrib("vertex", kHalf2_GrVertexAttribType); - fColor = &this->addVertexAttrib("color", kUByte4_norm_GrVertexAttribType); + fVertex = {"vertex", kHalf2_GrVertexAttribType}; + fColor = {"color", kUByte4_norm_GrVertexAttribType}; + this->setVertexAttributeCnt(2); } } const char* name() const override { return "GrMeshTest Processor"; } void getGLSLProcessorKey(const GrShaderCaps&, GrProcessorKeyBuilder* b) const final { - b->add32(SkToBool(fInstanceLocation)); - b->add32(SkToBool(fVertex)); + b->add32(fInstanceLocation.isInitialized()); + b->add32(fVertex.isInitialized()); } GrGLSLPrimitiveProcessor* createGLSLInstance(const GrShaderCaps&) const final; -protected: - const Attribute* fInstanceLocation; - const Attribute* fVertex; - const Attribute* fColor; +private: + const Attribute& onVertexAttribute(int i) const override { + if (fInstanceLocation.isInitialized()) { + return fVertex; + } + return IthAttribute(i, fVertex, fColor); + } + + const Attribute& onInstanceAttribute(int i) const override { + return IthAttribute(i, fInstanceLocation, fColor); + } + + Attribute fInstanceLocation; + Attribute fVertex; + Attribute fColor; friend class GLSLMeshTestProcessor; typedef GrGeometryProcessor INHERITED; @@ -338,15 +349,15 @@ class GLSLMeshTestProcessor : public GrGLSLGeometryProcessor { varyingHandler->addPassThroughAttribute(mp.fColor, args.fOutputColor); GrGLSLVertexBuilder* v = args.fVertBuilder; - if (!mp.fInstanceLocation) { - v->codeAppendf("float2 vertex = %s;", mp.fVertex->name()); + if (!mp.fInstanceLocation.isInitialized()) { + v->codeAppendf("float2 vertex = %s;", mp.fVertex.name()); } else { - if (mp.fVertex) { - v->codeAppendf("float2 offset = %s;", mp.fVertex->name()); + if (mp.fVertex.isInitialized()) { + v->codeAppendf("float2 offset = %s;", mp.fVertex.name()); } else { v->codeAppend ("float2 offset = float2(sk_VertexID / 2, sk_VertexID % 2);"); } - v->codeAppendf("float2 vertex = %s + offset * %i;", mp.fInstanceLocation->name(), + v->codeAppendf("float2 vertex = %s + offset * %i;", mp.fInstanceLocation.name(), kBoxSize); } gpArgs->fPositionVar.set(kFloat2_GrSLType, "vertex"); -- cgit v1.2.3