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

Unified Diff: src/gpu/GrOptDrawState.cpp

Issue 582963002: Solo gp (Closed) Base URL: https://skia.googlesource.com/skia.git@no_peb
Patch Set: fix Created 6 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: src/gpu/GrOptDrawState.cpp
diff --git a/src/gpu/GrOptDrawState.cpp b/src/gpu/GrOptDrawState.cpp
index cb03d2d683005dfe063bbedf49cf0cd53b8e933a..e84fbec8a3e12ca0e885bce765ec233ea3a074bb 100644
--- a/src/gpu/GrOptDrawState.cpp
+++ b/src/gpu/GrOptDrawState.cpp
@@ -36,7 +36,7 @@ GrOptDrawState::GrOptDrawState(const GrDrawState& drawState,
fInputCoverageIsUsed = true;
if (drawState.hasGeometryProcessor()) {
- fGeometryProcessor.reset(SkNEW_ARGS(GrEffectStage, (*drawState.getGeometryProcessor())));
+ fGeometryProcessor.reset(SkNEW_ARGS(GrGeometryStage, (*drawState.getGeometryProcessor())));
} else {
fGeometryProcessor.reset(NULL);
}
@@ -133,12 +133,12 @@ void GrOptDrawState::copyEffectiveColorStages(const GrDrawState& ds) {
}
for (int i = 0; i < ds.numColorStages(); ++i) {
- const GrEffect* effect = ds.getColorStage(i).getEffect();
- if (!effect->willUseInputColor()) {
+ const GrFragmentProcessor* fp = ds.getColorStage(i).getFragmentProcessor();
+ if (!fp->willUseInputColor()) {
firstColorStage = i;
fInputColorIsUsed = false;
}
- effect->getConstantColorComponents(&color, &validComponentFlags);
+ fp->getConstantColorComponents(&color, &validComponentFlags);
if (kRGBA_GrColorComponentFlags == validComponentFlags) {
firstColorStage = i + 1;
fColor = color;
@@ -165,8 +165,8 @@ void GrOptDrawState::copyEffectiveCoverageStages(const GrDrawState& ds) {
// input coverage in an effect
#ifdef OptCoverageStages
for (int i = 0; i < ds.numCoverageStages(); ++i) {
- const GrEffect* effect = ds.getCoverageStage(i).getEffect();
- if (!effect->willUseInputColor()) {
+ const GrProcessor* processor = ds.getCoverageStage(i).getProcessor();
+ if (!processor->willUseInputColor()) {
firstCoverageStage = i;
fInputCoverageIsUsed = false;
}
@@ -180,14 +180,15 @@ void GrOptDrawState::copyEffectiveCoverageStages(const GrDrawState& ds) {
}
}
-static void get_stage_stats(const GrEffectStage& stage, bool* readsDst, bool* readsFragPosition) {
- if (stage.getEffect()->willReadDstColor()) {
+static void get_stage_stats(const GrFragmentStage& stage, bool* readsDst, bool* readsFragPosition) {
+ if (stage.getFragmentProcessor()->willReadDstColor()) {
*readsDst = true;
}
- if (stage.getEffect()->willReadFragmentPosition()) {
+ if (stage.getFragmentProcessor()->willReadFragmentPosition()) {
*readsFragPosition = true;
}
}
+
void GrOptDrawState::getStageStats() {
// We will need a local coord attrib if there is one currently set on the optState and we are
// actually generating some effect code
@@ -201,16 +202,16 @@ void GrOptDrawState::getStageStats() {
fReadsFragPosition = false;
for (int s = 0; s < this->numColorStages(); ++s) {
- const GrEffectStage& stage = this->getColorStage(s);
+ const GrFragmentStage& stage = this->getColorStage(s);
get_stage_stats(stage, &fReadsDst, &fReadsFragPosition);
}
for (int s = 0; s < this->numCoverageStages(); ++s) {
- const GrEffectStage& stage = this->getCoverageStage(s);
+ const GrFragmentStage& stage = this->getCoverageStage(s);
get_stage_stats(stage, &fReadsDst, &fReadsFragPosition);
}
if (this->hasGeometryProcessor()) {
- const GrEffectStage& stage = *this->getGeometryProcessor();
- get_stage_stats(stage, &fReadsDst, &fReadsFragPosition);
+ const GrGeometryStage& stage = *this->getGeometryProcessor();
+ fReadsFragPosition = fReadsFragPosition || stage.getProcessor()->willReadFragmentPosition();
SkASSERT(fRequiresVertexShader);
}
}

Powered by Google App Engine
This is Rietveld 408576698