Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(203)

Side by Side Diff: src/gpu/GrPipeline.cpp

Issue 956363003: Cleanup in GrPipelineBuilder (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: fix Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/gpu/GrPipeline.h ('k') | src/gpu/GrPipelineBuilder.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2015 Google Inc. 2 * Copyright 2015 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "GrPipeline.h" 8 #include "GrPipeline.h"
9 9
10 #include "GrBatch.h" 10 #include "GrBatch.h"
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 73
74 GrXferProcessor::BlendInfo blendInfo; 74 GrXferProcessor::BlendInfo blendInfo;
75 fXferProcessor->getBlendInfo(&blendInfo); 75 fXferProcessor->getBlendInfo(&blendInfo);
76 76
77 this->adjustProgramFromOptimizations(pipelineBuilder, optFlags, colorPOI, co veragePOI, 77 this->adjustProgramFromOptimizations(pipelineBuilder, optFlags, colorPOI, co veragePOI,
78 &firstColorStageIdx, &firstCoverageStag eIdx); 78 &firstColorStageIdx, &firstCoverageStag eIdx);
79 79
80 bool usesLocalCoords = false; 80 bool usesLocalCoords = false;
81 81
82 // Copy Stages from PipelineBuilder to Pipeline 82 // Copy Stages from PipelineBuilder to Pipeline
83 for (int i = firstColorStageIdx; i < pipelineBuilder.numColorStages(); ++i) { 83 for (int i = firstColorStageIdx; i < pipelineBuilder.numColorFragmentStages( ); ++i) {
84 SkNEW_APPEND_TO_TARRAY(&fFragmentStages, 84 SkNEW_APPEND_TO_TARRAY(&fFragmentStages,
85 GrPendingFragmentStage, 85 GrPendingFragmentStage,
86 (pipelineBuilder.fColorStages[i])); 86 (pipelineBuilder.fColorStages[i]));
87 usesLocalCoords = usesLocalCoords || 87 usesLocalCoords = usesLocalCoords ||
88 pipelineBuilder.fColorStages[i].processor()->usesLocal Coords(); 88 pipelineBuilder.fColorStages[i].processor()->usesLocal Coords();
89 } 89 }
90 90
91 fNumColorStages = fFragmentStages.count(); 91 fNumColorStages = fFragmentStages.count();
92 for (int i = firstCoverageStageIdx; i < pipelineBuilder.numCoverageStages(); ++i) { 92 for (int i = firstCoverageStageIdx; i < pipelineBuilder.numCoverageFragmentS tages(); ++i) {
93 SkNEW_APPEND_TO_TARRAY(&fFragmentStages, 93 SkNEW_APPEND_TO_TARRAY(&fFragmentStages,
94 GrPendingFragmentStage, 94 GrPendingFragmentStage,
95 (pipelineBuilder.fCoverageStages[i])); 95 (pipelineBuilder.fCoverageStages[i]));
96 usesLocalCoords = usesLocalCoords || 96 usesLocalCoords = usesLocalCoords ||
97 pipelineBuilder.fCoverageStages[i].processor()->usesLo calCoords(); 97 pipelineBuilder.fCoverageStages[i].processor()->usesLo calCoords();
98 } 98 }
99 99
100 // let the GP init the batch tracker 100 // let the GP init the batch tracker
101 fInitBT.fColorIgnored = SkToBool(optFlags & GrXferProcessor::kIgnoreColor_Op tFlag); 101 fInitBT.fColorIgnored = SkToBool(optFlags & GrXferProcessor::kIgnoreColor_Op tFlag);
102 fInitBT.fOverrideColor = fInitBT.fColorIgnored ? GrColor_ILLEGAL : overrideC olor; 102 fInitBT.fOverrideColor = fInitBT.fColorIgnored ? GrColor_ILLEGAL : overrideC olor;
103 fInitBT.fCoverageIgnored = SkToBool(optFlags & GrXferProcessor::kIgnoreCover age_OptFlag); 103 fInitBT.fCoverageIgnored = SkToBool(optFlags & GrXferProcessor::kIgnoreCover age_OptFlag);
104 fInitBT.fUsesLocalCoords = usesLocalCoords; 104 fInitBT.fUsesLocalCoords = usesLocalCoords;
105 fInitBT.fCanTweakAlphaForCoverage = 105 fInitBT.fCanTweakAlphaForCoverage =
106 SkToBool(optFlags & GrXferProcessor::kCanTweakAlphaForCoverage_OptFlag); 106 SkToBool(optFlags & GrXferProcessor::kCanTweakAlphaForCoverage_OptFlag);
107 } 107 }
108 108
109 void GrPipeline::adjustProgramFromOptimizations(const GrPipelineBuilder& pipelin eBuilder, 109 void GrPipeline::adjustProgramFromOptimizations(const GrPipelineBuilder& pipelin eBuilder,
110 GrXferProcessor::OptFlags flags, 110 GrXferProcessor::OptFlags flags,
111 const GrProcOptInfo& colorPOI, 111 const GrProcOptInfo& colorPOI,
112 const GrProcOptInfo& coveragePOI , 112 const GrProcOptInfo& coveragePOI ,
113 int* firstColorStageIdx, 113 int* firstColorStageIdx,
114 int* firstCoverageStageIdx) { 114 int* firstCoverageStageIdx) {
115 fReadsFragPosition = false; 115 fReadsFragPosition = false;
116 116
117 if ((flags & GrXferProcessor::kIgnoreColor_OptFlag) || 117 if ((flags & GrXferProcessor::kIgnoreColor_OptFlag) ||
118 (flags & GrXferProcessor::kOverrideColor_OptFlag)) { 118 (flags & GrXferProcessor::kOverrideColor_OptFlag)) {
119 *firstColorStageIdx = pipelineBuilder.numColorStages(); 119 *firstColorStageIdx = pipelineBuilder.numColorFragmentStages();
120 } else { 120 } else {
121 fReadsFragPosition = colorPOI.readsFragPosition(); 121 fReadsFragPosition = colorPOI.readsFragPosition();
122 } 122 }
123 123
124 if (flags & GrXferProcessor::kIgnoreCoverage_OptFlag) { 124 if (flags & GrXferProcessor::kIgnoreCoverage_OptFlag) {
125 *firstCoverageStageIdx = pipelineBuilder.numCoverageStages(); 125 *firstCoverageStageIdx = pipelineBuilder.numCoverageFragmentStages();
126 } else { 126 } else {
127 if (coveragePOI.readsFragPosition()) { 127 if (coveragePOI.readsFragPosition()) {
128 fReadsFragPosition = true; 128 fReadsFragPosition = true;
129 } 129 }
130 } 130 }
131 } 131 }
132 132
133 //////////////////////////////////////////////////////////////////////////////// 133 ////////////////////////////////////////////////////////////////////////////////
134 134
135 bool GrPipeline::isEqual(const GrPipeline& that) const { 135 bool GrPipeline::isEqual(const GrPipeline& that) const {
(...skipping 15 matching lines...) Expand all
151 SkASSERT(this->numFragmentStages() == that.numFragmentStages()); 151 SkASSERT(this->numFragmentStages() == that.numFragmentStages());
152 for (int i = 0; i < this->numFragmentStages(); i++) { 152 for (int i = 0; i < this->numFragmentStages(); i++) {
153 153
154 if (this->getFragmentStage(i) != that.getFragmentStage(i)) { 154 if (this->getFragmentStage(i) != that.getFragmentStage(i)) {
155 return false; 155 return false;
156 } 156 }
157 } 157 }
158 return true; 158 return true;
159 } 159 }
160 160
OLDNEW
« no previous file with comments | « src/gpu/GrPipeline.h ('k') | src/gpu/GrPipelineBuilder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698