bff24abab8
It is difficult to do this both efficiently and correctly while honoring GLSL semantics (which require the lvalues to be kept distinct, even when they point to the same variable). We could make it work by making copies of every out parameter in each direction (going in for inouts, and coming out for outs and inouts). However, this could be self-defeating if it makes it harder for the driver to track variable lifetimes. Simply opting out of inlining these functions entirely seems like the best tradeoff; let the driver optimize them if it can, and we can enjoy reduced complexity in the SkSL inliner. Change-Id: I62f7b4550cc181cfe789e4f2ff4e408ba1baf9cb Bug: skia:11326 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/370257 Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: John Stiles <johnstiles@google.com> |
||
---|---|---|
.. | ||
DoWhileBodyMustBeInlinedIntoAScope.sksl | ||
DoWhileTestCannotBeInlined.sksl | ||
EnumsCanBeInlinedSafely.sksl | ||
ExponentialGrowth.sksl | ||
ForBodyMustBeInlinedIntoAScope.sksl | ||
ForInitializerExpressionsCanBeInlined.sksl | ||
ForWithoutReturnInsideCanBeInlined.sksl | ||
ForWithReturnInsideCannotBeInlined.sksl | ||
IfBodyMustBeInlinedIntoAScope.sksl | ||
IfElseBodyMustBeInlinedIntoAScope.sksl | ||
IfElseChainWithReturnsCanBeInlined.sksl | ||
IfTestCanBeInlined.sksl | ||
IfWithReturnsCanBeInlined.sksl | ||
InlineKeywordOverridesThreshold.sksl | ||
InlinerAvoidsVariableNameOverlap.sksl | ||
InlinerCanBeDisabled.sksl | ||
InlinerElidesTempVarForReturnsInsideBlock.sksl | ||
InlinerHonorsGLSLOutParamSemantics.sksl | ||
InlinerManglesNames.sksl | ||
InlinerUsesTempVarForMultipleReturns.sksl | ||
InlinerUsesTempVarForReturnsInsideBlockWithVar.sksl | ||
InlineThreshold.sksl | ||
InlineWithInoutArgument.sksl | ||
InlineWithModifiedArgument.sksl | ||
InlineWithNestedBigCalls.sksl | ||
InlineWithNestedCalls.sksl | ||
InlineWithUnmodifiedArgument.sksl | ||
InlineWithUnnecessaryBlocks.sksl | ||
NoInline.sksl | ||
ShortCircuitEvaluationsCannotInlineRightHandSide.sksl | ||
StaticSwitch.sksl | ||
StructsCanBeInlinedSafely.sksl | ||
SwitchWithCastCanBeInlined.sksl | ||
SwitchWithoutReturnInsideCanBeInlined.sksl | ||
SwizzleCanBeInlinedDirectly.sksl | ||
TernaryResultsCannotBeInlined.sksl | ||
TernaryTestCanBeInlined.sksl | ||
TrivialArgumentsInlineDirectly.sksl | ||
WhileBodyMustBeInlinedIntoAScope.sksl | ||
WhileTestCannotBeInlined.sksl |