commit de3d44109a504404f5eb3c2eef74b05ae2f00926
author Brian Salomon <bsalomon@google.com> Thu Sep 10 16:00:16 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Sep 11 19:06:34 2020 +0000
tree 126b039e6f87348c2de5959988642744bf3244af
parent b6b7fffc35020c205945d1c64a9201a0464984ef

Always make atlas texture index varying a float

This improves perf on ANGLE D3D11 ES3. Will make conditional
if any other config regresses.

Also makes the varyings passed to append_index_uv_varyings be
purely output params.


Change-Id: I332940d9d55be854d3bba72286ca89840ab43d50
Bug: skia:10644
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/316226
Reviewed-by: Jim Van Verth <jvanverth@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>

src/gpu/effects/GrBitmapTextGeoProc.cpp

diff --git a/src/gpu/effects/GrBitmapTextGeoProc.cpp b/src/gpu/effects/GrBitmapTextGeoProc.cpp
index f960540..bfa90c0 100644
--- a/src/gpu/effects/GrBitmapTextGeoProc.cpp
+++ b/src/gpu/effects/GrBitmapTextGeoProc.cpp
@@ -39,9 +39,7 @@
         fAtlasDimensionsInvUniform = uniformHandler->addUniform(nullptr, kVertex_GrShaderFlag,
                 kFloat2_GrSLType, "AtlasSizeInv", &atlasDimensionsInvName);
 
-        GrGLSLVarying uv(kFloat2_GrSLType);
-        GrSLType texIdxType = args.fShaderCaps->integerSupport() ? kInt_GrSLType : kFloat_GrSLType;
-        GrGLSLVarying texIdx(texIdxType);
+        GrGLSLVarying uv, texIdx;
         append_index_uv_varyings(args, btgp.numTextureSamplers(), btgp.inTextureCoords().name(),
                                  atlasDimensionsInvName, &uv, &texIdx, nullptr);