commit ad22300d6c385377b32b68360c8d65355a8c3298
author John Stiles <johnstiles@google.com> Fri Mar 12 17:33:17 2021 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Sat Mar 13 13:18:47 2021 +0000
tree e8750db550f9ee9bd64350eaf4095364e9964bdf
parent 46e7507b0daf34ca967e5888832b22d2d712fe0e

Remove `fForceInline` argument from EmitArgs.

This improved codegen when it was first introduced. Since that time, our
inliner has become much smarter and no longer needs these hints.

Change-Id: I7725da1eb9814e656849881ad2ca2b1bc038e4a0
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/384320
Auto-Submit: John Stiles <johnstiles@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>

src/gpu/glsl/GrGLSLFragmentShaderBuilder.cpp

diff --git a/src/gpu/glsl/GrGLSLFragmentShaderBuilder.cpp b/src/gpu/glsl/GrGLSLFragmentShaderBuilder.cpp
index bb9741d..aed5fea 100644
--- a/src/gpu/glsl/GrGLSLFragmentShaderBuilder.cpp
+++ b/src/gpu/glsl/GrGLSLFragmentShaderBuilder.cpp
@@ -149,7 +149,7 @@
 
     SkString funcName = this->getMangledFunctionName(args.fFp.name());
     this->emitFunction(kHalf4_GrSLType, funcName.c_str(), {params, paramCount},
-                       this->code().c_str(), args.fForceInline);
+                       this->code().c_str());
     this->deleteStage();
     this->onAfterChildProcEmitCode();
     return funcName;