Brian Salomon | 92ce594 | 2017-01-18 11:01:10 -0500 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2017 Google Inc. |
| 3 | * |
| 4 | * Use of this source code is governed by a BSD-style license that can be |
| 5 | * found in the LICENSE file. |
| 6 | */ |
| 7 | |
| 8 | #include "GrProcessorSet.h" |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 9 | #include "GrAppliedClip.h" |
| 10 | #include "GrCaps.h" |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 11 | #include "GrXferProcessor.h" |
Brian Salomon | 477d0ef | 2017-07-14 10:12:26 -0400 | [diff] [blame] | 12 | #include "SkBlendModePriv.h" |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 13 | #include "effects/GrPorterDuffXferProcessor.h" |
Brian Salomon | 92ce594 | 2017-01-18 11:01:10 -0500 | [diff] [blame] | 14 | |
Brian Salomon | 292bf7a | 2017-05-17 09:43:55 -0400 | [diff] [blame] | 15 | const GrProcessorSet& GrProcessorSet::EmptySet() { |
Brian Salomon | 91326c3 | 2017-08-09 16:02:19 -0400 | [diff] [blame] | 16 | static GrProcessorSet gEmpty(GrProcessorSet::Empty::kEmpty); |
Brian Salomon | 292bf7a | 2017-05-17 09:43:55 -0400 | [diff] [blame] | 17 | return gEmpty; |
| 18 | } |
Brian Salomon | 6d4b65e | 2017-05-03 17:06:09 -0400 | [diff] [blame] | 19 | |
Brian Salomon | 91326c3 | 2017-08-09 16:02:19 -0400 | [diff] [blame] | 20 | GrProcessorSet GrProcessorSet::MakeEmptySet() { |
| 21 | return GrProcessorSet(GrProcessorSet::Empty::kEmpty); |
| 22 | } |
| 23 | |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 24 | GrProcessorSet::GrProcessorSet(GrPaint&& paint) : fXP(paint.getXPFactory()) { |
Brian Salomon | f87e2b9 | 2017-01-19 11:31:50 -0500 | [diff] [blame] | 25 | fFlags = 0; |
Brian Salomon | 8d2f90b | 2017-03-13 09:11:58 -0400 | [diff] [blame] | 26 | if (paint.numColorFragmentProcessors() <= kMaxColorProcessors) { |
| 27 | fColorFragmentProcessorCnt = paint.numColorFragmentProcessors(); |
| 28 | fFragmentProcessors.reset(paint.numTotalFragmentProcessors()); |
| 29 | int i = 0; |
| 30 | for (auto& fp : paint.fColorFragmentProcessors) { |
Brian Salomon | 5f970fe | 2017-06-16 17:30:59 -0400 | [diff] [blame] | 31 | SkASSERT(fp.get()); |
Brian Salomon | aff329b | 2017-08-11 09:40:37 -0400 | [diff] [blame] | 32 | fFragmentProcessors[i++] = std::move(fp); |
Brian Salomon | 8d2f90b | 2017-03-13 09:11:58 -0400 | [diff] [blame] | 33 | } |
| 34 | for (auto& fp : paint.fCoverageFragmentProcessors) { |
Brian Salomon | 5f970fe | 2017-06-16 17:30:59 -0400 | [diff] [blame] | 35 | SkASSERT(fp.get()); |
Brian Salomon | aff329b | 2017-08-11 09:40:37 -0400 | [diff] [blame] | 36 | fFragmentProcessors[i++] = std::move(fp); |
Brian Salomon | 8d2f90b | 2017-03-13 09:11:58 -0400 | [diff] [blame] | 37 | } |
Brian Salomon | 8d2f90b | 2017-03-13 09:11:58 -0400 | [diff] [blame] | 38 | } else { |
| 39 | SkDebugf("Insane number of color fragment processors in paint. Dropping all processors."); |
| 40 | fColorFragmentProcessorCnt = 0; |
Brian Salomon | f87e2b9 | 2017-01-19 11:31:50 -0500 | [diff] [blame] | 41 | } |
Brian Salomon | 92ce594 | 2017-01-18 11:01:10 -0500 | [diff] [blame] | 42 | } |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 43 | |
Brian Salomon | 477d0ef | 2017-07-14 10:12:26 -0400 | [diff] [blame] | 44 | GrProcessorSet::GrProcessorSet(SkBlendMode mode) |
| 45 | : fXP(SkBlendMode_AsXPFactory(mode)) |
| 46 | , fColorFragmentProcessorCnt(0) |
| 47 | , fFragmentProcessorOffset(0) |
| 48 | , fFlags(0) {} |
| 49 | |
Brian Salomon | aff329b | 2017-08-11 09:40:37 -0400 | [diff] [blame] | 50 | GrProcessorSet::GrProcessorSet(std::unique_ptr<GrFragmentProcessor> colorFP) |
Brian Salomon | 82ddc94 | 2017-07-14 12:00:13 -0400 | [diff] [blame] | 51 | : fFragmentProcessors(1) |
| 52 | , fXP((const GrXPFactory*)nullptr) |
| 53 | , fColorFragmentProcessorCnt(1) |
| 54 | , fFragmentProcessorOffset(0) |
| 55 | , fFlags(0) { |
| 56 | SkASSERT(colorFP); |
Brian Salomon | aff329b | 2017-08-11 09:40:37 -0400 | [diff] [blame] | 57 | fFragmentProcessors[0] = std::move(colorFP); |
Brian Salomon | 82ddc94 | 2017-07-14 12:00:13 -0400 | [diff] [blame] | 58 | } |
| 59 | |
Brian Salomon | 91326c3 | 2017-08-09 16:02:19 -0400 | [diff] [blame] | 60 | GrProcessorSet::GrProcessorSet(GrProcessorSet&& that) |
| 61 | : fXP(std::move(that.fXP)) |
| 62 | , fColorFragmentProcessorCnt(that.fColorFragmentProcessorCnt) |
| 63 | , fFragmentProcessorOffset(0) |
| 64 | , fFlags(that.fFlags) { |
| 65 | fFragmentProcessors.reset(that.fFragmentProcessors.count() - that.fFragmentProcessorOffset); |
| 66 | for (int i = 0; i < fFragmentProcessors.count(); ++i) { |
Brian Salomon | aff329b | 2017-08-11 09:40:37 -0400 | [diff] [blame] | 67 | fFragmentProcessors[i] = |
| 68 | std::move(that.fFragmentProcessors[i + that.fFragmentProcessorOffset]); |
Brian Salomon | 91326c3 | 2017-08-09 16:02:19 -0400 | [diff] [blame] | 69 | } |
| 70 | that.fColorFragmentProcessorCnt = 0; |
| 71 | that.fFragmentProcessors.reset(0); |
| 72 | } |
| 73 | |
Brian Salomon | 54d212e | 2017-03-21 14:22:38 -0400 | [diff] [blame] | 74 | GrProcessorSet::~GrProcessorSet() { |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 75 | if (this->isFinalized() && this->xferProcessor()) { |
Brian Salomon | d61c9d9 | 2017-04-10 10:54:25 -0400 | [diff] [blame] | 76 | this->xferProcessor()->unref(); |
Brian Salomon | 54d212e | 2017-03-21 14:22:38 -0400 | [diff] [blame] | 77 | } |
| 78 | } |
| 79 | |
Brian Salomon | 82dfd3d | 2017-06-14 12:30:35 -0400 | [diff] [blame] | 80 | SkString dump_fragment_processor_tree(const GrFragmentProcessor* fp, int indentCnt) { |
| 81 | SkString result; |
| 82 | SkString indentString; |
| 83 | for (int i = 0; i < indentCnt; ++i) { |
| 84 | indentString.append(" "); |
| 85 | } |
| 86 | result.appendf("%s%s %s \n", indentString.c_str(), fp->name(), fp->dumpInfo().c_str()); |
| 87 | if (fp->numChildProcessors()) { |
| 88 | for (int i = 0; i < fp->numChildProcessors(); ++i) { |
| 89 | result += dump_fragment_processor_tree(&fp->childProcessor(i), indentCnt + 1); |
| 90 | } |
| 91 | } |
| 92 | return result; |
| 93 | } |
| 94 | |
| 95 | SkString GrProcessorSet::dumpProcessors() const { |
| 96 | SkString result; |
| 97 | if (this->numFragmentProcessors()) { |
| 98 | if (this->numColorFragmentProcessors()) { |
| 99 | result.append("Color Fragment Processors:\n"); |
| 100 | for (int i = 0; i < this->numColorFragmentProcessors(); ++i) { |
| 101 | result += dump_fragment_processor_tree(this->colorFragmentProcessor(i), 1); |
| 102 | } |
| 103 | } else { |
| 104 | result.append("No color fragment processors.\n"); |
| 105 | } |
| 106 | if (this->numCoverageFragmentProcessors()) { |
| 107 | result.append("Coverage Fragment Processors:\n"); |
| 108 | for (int i = 0; i < this->numColorFragmentProcessors(); ++i) { |
| 109 | result += dump_fragment_processor_tree(this->coverageFragmentProcessor(i), 1); |
| 110 | } |
| 111 | } else { |
| 112 | result.append("No coverage fragment processors.\n"); |
| 113 | } |
| 114 | } else { |
| 115 | result.append("No color or coverage fragment processors.\n"); |
| 116 | } |
| 117 | if (this->isFinalized()) { |
| 118 | result.append("Xfer Processor: "); |
| 119 | if (this->xferProcessor()) { |
| 120 | result.appendf("%s\n", this->xferProcessor()->name()); |
| 121 | } else { |
| 122 | result.append("SrcOver\n"); |
| 123 | } |
| 124 | } else { |
| 125 | result.append("XP Factory dumping not implemented.\n"); |
| 126 | } |
| 127 | return result; |
| 128 | } |
| 129 | |
Brian Salomon | 54d212e | 2017-03-21 14:22:38 -0400 | [diff] [blame] | 130 | bool GrProcessorSet::operator==(const GrProcessorSet& that) const { |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 131 | SkASSERT(this->isFinalized()); |
| 132 | SkASSERT(that.isFinalized()); |
Brian Salomon | 70288c0 | 2017-03-24 12:27:17 -0400 | [diff] [blame] | 133 | int fpCount = this->numFragmentProcessors(); |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 134 | if (((fFlags ^ that.fFlags) & ~kFinalized_Flag) || fpCount != that.numFragmentProcessors() || |
Brian Salomon | 54d212e | 2017-03-21 14:22:38 -0400 | [diff] [blame] | 135 | fColorFragmentProcessorCnt != that.fColorFragmentProcessorCnt) { |
| 136 | return false; |
| 137 | } |
Brian Salomon | 70288c0 | 2017-03-24 12:27:17 -0400 | [diff] [blame] | 138 | |
| 139 | for (int i = 0; i < fpCount; ++i) { |
| 140 | int a = i + fFragmentProcessorOffset; |
| 141 | int b = i + that.fFragmentProcessorOffset; |
| 142 | if (!fFragmentProcessors[a]->isEqual(*that.fFragmentProcessors[b])) { |
Brian Salomon | 54d212e | 2017-03-21 14:22:38 -0400 | [diff] [blame] | 143 | return false; |
| 144 | } |
| 145 | } |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 146 | // Most of the time both of these are null |
| 147 | if (!this->xferProcessor() && !that.xferProcessor()) { |
| 148 | return true; |
Brian Salomon | 54d212e | 2017-03-21 14:22:38 -0400 | [diff] [blame] | 149 | } |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 150 | const GrXferProcessor& thisXP = this->xferProcessor() |
| 151 | ? *this->xferProcessor() |
| 152 | : GrPorterDuffXPFactory::SimpleSrcOverXP(); |
| 153 | const GrXferProcessor& thatXP = that.xferProcessor() |
| 154 | ? *that.xferProcessor() |
| 155 | : GrPorterDuffXPFactory::SimpleSrcOverXP(); |
| 156 | return thisXP.isEqual(thatXP); |
Brian Salomon | 54d212e | 2017-03-21 14:22:38 -0400 | [diff] [blame] | 157 | } |
| 158 | |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 159 | GrProcessorSet::Analysis GrProcessorSet::finalize(const GrProcessorAnalysisColor& colorInput, |
| 160 | const GrProcessorAnalysisCoverage coverageInput, |
| 161 | const GrAppliedClip* clip, bool isMixedSamples, |
Brian Osman | 9a725dd | 2017-09-20 09:53:22 -0400 | [diff] [blame] | 162 | const GrCaps& caps, |
| 163 | GrPixelConfigIsClamped dstIsClamped, |
| 164 | GrColor* overrideInputColor) { |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 165 | SkASSERT(!this->isFinalized()); |
| 166 | SkASSERT(!fFragmentProcessorOffset); |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 167 | |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 168 | GrProcessorSet::Analysis analysis; |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 169 | analysis.fCompatibleWithCoverageAsAlpha = GrProcessorAnalysisCoverage::kLCD != coverageInput; |
| 170 | |
Brian Salomon | 650ced0 | 2017-07-20 16:46:46 -0400 | [diff] [blame] | 171 | const GrFragmentProcessor* clipFP = clip ? clip->clipCoverageFragmentProcessor() : nullptr; |
Brian Salomon | aff329b | 2017-08-11 09:40:37 -0400 | [diff] [blame] | 172 | const std::unique_ptr<const GrFragmentProcessor>* fps = |
| 173 | fFragmentProcessors.get() + fFragmentProcessorOffset; |
| 174 | GrColorFragmentProcessorAnalysis colorAnalysis( |
| 175 | colorInput, unique_ptr_address_as_pointer_address(fps), fColorFragmentProcessorCnt); |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 176 | analysis.fCompatibleWithCoverageAsAlpha &= |
| 177 | colorAnalysis.allProcessorsCompatibleWithCoverageAsAlpha(); |
| 178 | fps += fColorFragmentProcessorCnt; |
| 179 | int n = this->numCoverageFragmentProcessors(); |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 180 | bool hasCoverageFP = n > 0; |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 181 | bool coverageUsesLocalCoords = false; |
Brian Salomon | bfafcba | 2017-03-02 08:49:19 -0500 | [diff] [blame] | 182 | for (int i = 0; i < n; ++i) { |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 183 | if (!fps[i]->compatibleWithCoverageAsAlpha()) { |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 184 | analysis.fCompatibleWithCoverageAsAlpha = false; |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 185 | // Other than tests that exercise atypical behavior we expect all coverage FPs to be |
| 186 | // compatible with the coverage-as-alpha optimization. |
| 187 | GrCapsDebugf(&caps, "Coverage FP is not compatible with coverage as alpha.\n"); |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 188 | } |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 189 | coverageUsesLocalCoords |= fps[i]->usesLocalCoords(); |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 190 | } |
| 191 | |
| 192 | if (clipFP) { |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 193 | analysis.fCompatibleWithCoverageAsAlpha &= clipFP->compatibleWithCoverageAsAlpha(); |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 194 | coverageUsesLocalCoords |= clipFP->usesLocalCoords(); |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 195 | hasCoverageFP = true; |
| 196 | } |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 197 | int colorFPsToEliminate = colorAnalysis.initialProcessorsToEliminate(overrideInputColor); |
| 198 | analysis.fInputColorType = static_cast<Analysis::PackedInputColorType>( |
| 199 | colorFPsToEliminate ? Analysis::kOverridden_InputColorType |
| 200 | : Analysis::kOriginal_InputColorType); |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 201 | |
Brian Salomon | a811b12 | 2017-03-30 08:21:32 -0400 | [diff] [blame] | 202 | GrProcessorAnalysisCoverage outputCoverage; |
| 203 | if (GrProcessorAnalysisCoverage::kLCD == coverageInput) { |
| 204 | outputCoverage = GrProcessorAnalysisCoverage::kLCD; |
| 205 | } else if (hasCoverageFP || GrProcessorAnalysisCoverage::kSingleChannel == coverageInput) { |
| 206 | outputCoverage = GrProcessorAnalysisCoverage::kSingleChannel; |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 207 | } else { |
Brian Salomon | a811b12 | 2017-03-30 08:21:32 -0400 | [diff] [blame] | 208 | outputCoverage = GrProcessorAnalysisCoverage::kNone; |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 209 | } |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 210 | |
| 211 | GrXPFactory::AnalysisProperties props = GrXPFactory::GetAnalysisProperties( |
Brian Osman | 9a725dd | 2017-09-20 09:53:22 -0400 | [diff] [blame] | 212 | this->xpFactory(), colorAnalysis.outputColor(), outputCoverage, caps, dstIsClamped); |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 213 | if (!this->numCoverageFragmentProcessors() && |
Brian Salomon | a811b12 | 2017-03-30 08:21:32 -0400 | [diff] [blame] | 214 | GrProcessorAnalysisCoverage::kNone == coverageInput) { |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 215 | analysis.fCanCombineOverlappedStencilAndCover = SkToBool( |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 216 | props & GrXPFactory::AnalysisProperties::kCanCombineOverlappedStencilAndCover); |
| 217 | } else { |
| 218 | // If we have non-clipping coverage processors we don't try to merge stencil steps as its |
| 219 | // unclear whether it will be correct. We don't expect this to happen in practice. |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 220 | analysis.fCanCombineOverlappedStencilAndCover = false; |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 221 | } |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 222 | analysis.fRequiresDstTexture = |
| 223 | SkToBool(props & GrXPFactory::AnalysisProperties::kRequiresDstTexture); |
| 224 | analysis.fCompatibleWithCoverageAsAlpha &= |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 225 | SkToBool(props & GrXPFactory::AnalysisProperties::kCompatibleWithAlphaAsCoverage); |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 226 | analysis.fRequiresBarrierBetweenOverlappingDraws = SkToBool( |
Brian Salomon | 4fc7740 | 2017-03-30 16:48:26 -0400 | [diff] [blame] | 227 | props & GrXPFactory::AnalysisProperties::kRequiresBarrierBetweenOverlappingDraws); |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 228 | if (props & GrXPFactory::AnalysisProperties::kIgnoresInputColor) { |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 229 | colorFPsToEliminate = this->numColorFragmentProcessors(); |
| 230 | analysis.fInputColorType = |
| 231 | static_cast<Analysis::PackedInputColorType>(Analysis::kIgnored_InputColorType); |
| 232 | analysis.fUsesLocalCoords = coverageUsesLocalCoords; |
Brian Salomon | 3185384 | 2017-03-28 16:32:05 -0400 | [diff] [blame] | 233 | } else { |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 234 | analysis.fUsesLocalCoords = coverageUsesLocalCoords | colorAnalysis.usesLocalCoords(); |
Brian Salomon | 5298dc8 | 2017-02-22 11:52:03 -0500 | [diff] [blame] | 235 | } |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 236 | for (int i = 0; i < colorFPsToEliminate; ++i) { |
Brian Salomon | aff329b | 2017-08-11 09:40:37 -0400 | [diff] [blame] | 237 | fFragmentProcessors[i].reset(nullptr); |
Brian Salomon | 5dac9b3 | 2017-04-08 02:53:30 +0000 | [diff] [blame] | 238 | } |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 239 | for (int i = colorFPsToEliminate; i < fFragmentProcessors.count(); ++i) { |
Brian Salomon | aff329b | 2017-08-11 09:40:37 -0400 | [diff] [blame] | 240 | fFragmentProcessors[i]->markPendingExecution(); |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 241 | } |
| 242 | fFragmentProcessorOffset = colorFPsToEliminate; |
| 243 | fColorFragmentProcessorCnt -= colorFPsToEliminate; |
| 244 | |
| 245 | auto xp = GrXPFactory::MakeXferProcessor(this->xpFactory(), colorAnalysis.outputColor(), |
Brian Osman | 9a725dd | 2017-09-20 09:53:22 -0400 | [diff] [blame] | 246 | outputCoverage, isMixedSamples, caps, dstIsClamped); |
Brian Salomon | d61c9d9 | 2017-04-10 10:54:25 -0400 | [diff] [blame] | 247 | fXP.fProcessor = xp.release(); |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 248 | |
Brian Salomon | d61c9d9 | 2017-04-10 10:54:25 -0400 | [diff] [blame] | 249 | fFlags |= kFinalized_Flag; |
Brian Salomon | 48d1b4c | 2017-04-08 07:38:53 -0400 | [diff] [blame] | 250 | analysis.fIsInitialized = true; |
| 251 | return analysis; |
Brian Salomon | 70288c0 | 2017-03-24 12:27:17 -0400 | [diff] [blame] | 252 | } |