diff options
author | 2016-12-12 17:20:42 +0000 | |
---|---|---|
committer | 2016-12-12 17:22:28 +0000 | |
commit | e8e4a3e6782586680086a0279eafb89969c29f3d (patch) | |
tree | 93e27a37cd75c69fe85fab990f6daf17c80ed794 /src/gpu/vk/GrVkPipelineStateBuilder.cpp | |
parent | d737bee1470bbba8af5c9d74cbec2d731da33804 (diff) |
Revert "added sk_FragCoord support to skslc"
This reverts commit ce33f10677630e34187b661a02161378d8304d68.
Reason for revert: Breaking many gpu bots
Change-Id: I94c813ed6a9311458c872f74bb1b0792f46ff414
Reviewed-on: https://skia-review.googlesource.com/5737
Commit-Queue: Greg Daniel <egdaniel@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
Diffstat (limited to 'src/gpu/vk/GrVkPipelineStateBuilder.cpp')
-rw-r--r-- | src/gpu/vk/GrVkPipelineStateBuilder.cpp | 40 |
1 files changed, 14 insertions, 26 deletions
diff --git a/src/gpu/vk/GrVkPipelineStateBuilder.cpp b/src/gpu/vk/GrVkPipelineStateBuilder.cpp index 4125938bae..49f5cf8a0c 100644 --- a/src/gpu/vk/GrVkPipelineStateBuilder.cpp +++ b/src/gpu/vk/GrVkPipelineStateBuilder.cpp @@ -58,11 +58,11 @@ void GrVkPipelineStateBuilder::finalizeFragmentSecondaryColor(GrShaderVar& outpu outputColor.addLayoutQualifier("location = 0, index = 1"); } -bool GrVkPipelineStateBuilder::createVkShaderModule(VkShaderStageFlagBits stage, +bool GrVkPipelineStateBuilder::CreateVkShaderModule(const GrVkGpu* gpu, + VkShaderStageFlagBits stage, const GrGLSLShaderBuilder& builder, VkShaderModule* shaderModule, - VkPipelineShaderStageCreateInfo* stageInfo, - const SkSL::Program::Settings& settings) { + VkPipelineShaderStageCreateInfo* stageInfo) { SkString shaderString; for (int i = 0; i < builder.fCompilerStrings.count(); ++i) { if (builder.fCompilerStrings[i]) { @@ -70,17 +70,7 @@ bool GrVkPipelineStateBuilder::createVkShaderModule(VkShaderStageFlagBits stage, shaderString.append("\n"); } } - - SkSL::Program::Inputs inputs; - bool result = GrCompileVkShaderModule(fGpu, shaderString.c_str(), stage, shaderModule, - stageInfo, settings, &inputs); - if (!result) { - return false; - } - if (inputs.fRTHeight) { - this->addRTHeightUniform(SKSL_RTHEIGHT_NAME); - } - return result; + return GrCompileVkShaderModule(gpu, shaderString.c_str(), stage, shaderModule, stageInfo); } GrVkPipelineState* GrVkPipelineStateBuilder::finalize(const GrStencilSettings& stencil, @@ -127,22 +117,20 @@ GrVkPipelineState* GrVkPipelineStateBuilder::finalize(const GrStencilSettings& s this->finalizeShaders(); VkPipelineShaderStageCreateInfo shaderStageInfo[2]; - SkSL::Program::Settings settings; - settings.fFlipY = this->pipeline().getRenderTarget()->origin() != kTopLeft_GrSurfaceOrigin; - SkAssertResult(this->createVkShaderModule(VK_SHADER_STAGE_VERTEX_BIT, - fVS, - &vertShaderModule, - &shaderStageInfo[0], - settings)); + SkAssertResult(CreateVkShaderModule(fGpu, + VK_SHADER_STAGE_VERTEX_BIT, + fVS, + &vertShaderModule, + &shaderStageInfo[0])); // TODO: geometry shader support. SkASSERT(!this->primitiveProcessor().willUseGeoShader()); - SkAssertResult(this->createVkShaderModule(VK_SHADER_STAGE_FRAGMENT_BIT, - fFS, - &fragShaderModule, - &shaderStageInfo[1], - settings)); + SkAssertResult(CreateVkShaderModule(fGpu, + VK_SHADER_STAGE_FRAGMENT_BIT, + fFS, + &fragShaderModule, + &shaderStageInfo[1])); GrVkPipeline* pipeline = resourceProvider.createPipeline(fPipeline, stencil, |