commit a045eea29eb79254248d7e2666825a1ca7d23d4e
author Chris Dalton <csmartdalton@google.com> Tue Oct 24 13:22:10 2017 -0600
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Wed Oct 25 15:44:05 2017 +0000
tree 4a0a55d8476a100c1fa5b54ca5212a02723a148f
parent d204eb80abb0172b024664cf9f05d66d15574a24

CCPR: use 16-bit ints for the atlas offset attrib

Bug: skia:
Change-Id: I053e8416ced317b6ebc46cc8189840d60c7f91e8
Reviewed-on: https://skia-review.googlesource.com/62060
Commit-Queue: Chris Dalton <csmartdalton@google.com>
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>

src/gpu/ccpr/GrCCPRPathProcessor.cpp

diff --git a/src/gpu/ccpr/GrCCPRPathProcessor.cpp b/src/gpu/ccpr/GrCCPRPathProcessor.cpp
index 6608ce81..ff3c60c 100644
--- a/src/gpu/ccpr/GrCCPRPathProcessor.cpp
+++ b/src/gpu/ccpr/GrCCPRPathProcessor.cpp
@@ -56,9 +56,7 @@
     this->addInstanceAttrib("devbounds45", kFloat4_GrVertexAttribType);
     this->addInstanceAttrib("view_matrix", kFloat4_GrVertexAttribType);
     this->addInstanceAttrib("view_translate", kFloat2_GrVertexAttribType);
-    // FIXME: this could be a vector of two shorts if it were supported by Ganesh.
-    // Note: this should be doable now with kUShort2_GrVertexAttribType
-    this->addInstanceAttrib("atlas_offset", kInt2_GrVertexAttribType);
+    this->addInstanceAttrib("atlas_offset", kShort2_GrVertexAttribType);
     this->addInstanceAttrib("color", kUByte4_norm_GrVertexAttribType);
 
     SkASSERT(offsetof(Instance, fDevBounds) ==